Skip to content

Obsolete the for Sized? syntax #20556

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Jan 6, 2015
Merged

Obsolete the for Sized? syntax #20556

merged 6 commits into from
Jan 6, 2015

Conversation

japaric
Copy link
Member

@japaric japaric commented Jan 5, 2015

Traits and their Self type no longer have an implicit Sized bound.

If you were using the for Sized? syntax simply remove it from your trait:

// change this
trait Foo for Sized? {}
// into this
trait Foo {}

[breaking-change]

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Jan 5, 2015
Conflicts:
	src/libserialize/serialize.rs
	src/libstd/c_str.rs
@japaric japaric changed the title Remove uses of for Sized? on traits Obsolete the for Sized? syntax Jan 5, 2015
@alexcrichton
Copy link
Member

I've merged this into a pending rollup (if you want to hold off on a rebase)

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Jan 6, 2015
Conflicts:
	src/libcollections/slice.rs
	src/libcollections/str.rs
	src/libcore/borrow.rs
	src/libcore/cmp.rs
	src/libcore/ops.rs
	src/libstd/c_str.rs
	src/test/compile-fail/issue-19009.rs
@bors bors merged commit cd4205a into rust-lang:master Jan 6, 2015
@japaric japaric deleted the no-for-sized branch January 22, 2015 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants