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

Rollup of 11 pull requests #27265

Merged
merged 28 commits into from
Jul 24, 2015
Merged

Rollup of 11 pull requests #27265

merged 28 commits into from
Jul 24, 2015

Conversation

aidanhs and others added 28 commits July 21, 2015 19:07
…excrichton

fixes the linking error on bitrig.
Was browsing somebody else's code and came across a snippet using labels. Looking around, it seems like there was an example for this in [rustbyexample](http://rustbyexample.com/flow_control/loop/nested.html) but none in trpl.
…arth

As title!

I should probably be bunching these up a bit more, but I'm not sure when my time is going to disappear on me.  Once my schedule stabilises I'll try to start batching them into larger PRs.

Part of rust-lang#24407.
r? @Manishearth
Hi all.
This is my first contribution to Rust and fixes an issue causing an invalid error message to be presented to the user when using unit struct as length of a repeat expression, issue rust-lang#27008. The solution is based on suggestions by @oli-obk, but as I'm a complete newbie to this, I have no clue if I got them right :)
The biggest concern I have is that if the `NodeId` I'm returning is the correct one or not (it's not meaningful in this case but I think it would be nice to get it right).
@steveklabnik
Copy link
Member Author

@bors: r+ p=10000

@bors
Copy link
Contributor

bors commented Jul 24, 2015

📌 Commit 427238c has been approved by steveklabnik

@rust-highfive
Copy link
Collaborator

r? @huonw

(rust_highfive has picked a reviewer for you, use r? to override)

bors added a commit that referenced this pull request Jul 24, 2015
@bors
Copy link
Contributor

bors commented Jul 24, 2015

⌛ Testing commit 427238c with merge 82d40cb...

@bors bors merged commit 427238c into rust-lang:master Jul 24, 2015
@Centril Centril added the rollup A PR which is a rollup label Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet