Skip to content
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

Implement the new for-loop protocol #6223

Merged
merged 9 commits into from
May 11, 2013
Merged

Conversation

alexcrichton
Copy link
Member

Closes #6183.

The first commit changes the compiler's method of treating a for loop, and all the remaining commits are just dealing with the fallout.

The biggest fallout was the IterBytes trait, although it's really a whole lot nicer now because all of the iter_bytes_XX methods are just and-ed together. Sadly there was a huge amount of stuff that's cfg(stage0) gated, but whoever lands the next snapshot is going to have a lot of fun deleting all this code!

@brson
Copy link
Contributor

brson commented May 3, 2013

Exciting! 🚴

@graydon
Copy link
Contributor

graydon commented May 7, 2013

needs a rebase, but I'm into it! thanks :)

@alexcrichton
Copy link
Member Author

Should be all good to go now.

@alexcrichton
Copy link
Member Author

I thought I had run all the tests, but apparently not. I've got make check running now, but I'm hopeful that this is ready to go in the meantime.

bors added a commit that referenced this pull request May 11, 2013
Closes #6183.

The first commit changes the compiler's method of treating a `for` loop, and all the remaining commits are just dealing with the fallout.

The biggest fallout was the `IterBytes` trait, although it's really a whole lot nicer now because all of the `iter_bytes_XX` methods are just and-ed together. Sadly there was a huge amount of stuff that's `cfg(stage0)` gated, but whoever lands the next snapshot is going to have a lot of fun deleting all this code!
@bors bors closed this May 11, 2013
@bors bors merged commit 606bd75 into rust-lang:incoming May 11, 2013
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