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 7 pull requests #25262

Merged
merged 18 commits into from
May 10, 2015
Merged

Rollup of 7 pull requests #25262

merged 18 commits into from
May 10, 2015

Conversation

carols10cents and others added 18 commits May 9, 2015 13:10
Remove an rogue 'is' and fix some line wrapping.
In my opinion this looks nicer, but also it matches the whitespace generally
used for stability markers more closely.
If the user intended to set the crate_type to "lib" but accidentally used
incorrect syntax such as `#![crate_type(lib)]`, the compilation would fail with
"main function not found". This made it hard to locate the source of the
problem, since the failure would cause the warning about the incorrect
attribute not to be shown.
Fix two typos while `io::stdin().read_line()` returns `Result` actually

Signed-off-by: acgtyrant <acgtyrant@gmail.com>
Fix a typo while `io::stdin().read_line()` returns `Result` actually
…=Manishearth

Hiiiii Rust, it's me again, Carol ;)

Based on the unused files that I found hanging out and that I removed in rust-lang#25231, I looked at the rest of the auxiliary files to see if there were any more not being used. Guess what!! There were!!!!! ;)

Have I mentioned how much I enjoy deleting code? ;) ;) ;) <3
Remove an rogue 'is' and fix some line wrapping.
…hearth

In my opinion this looks nicer, but also it matches the whitespace generally
used for stability markers.
… r=alexcrichton

Fixes the problem in rust-lang#16974 with unhelpful error messages when accidentally using the wrong syntax for the `crate_type="lib"` attribute. The attribute syntax error now shows up instead of "main function not found".
…r=alexcrichton

This was added in d2f8074 along with the AssocSpace stuff. The AssocSpace
stuff was then removed in de8e0ae, except it seems the comment here was missed.
@rust-highfive
Copy link
Collaborator

r? @pcwalton

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

@Manishearth
Copy link
Member Author

@bors: r+ force

@bors
Copy link
Contributor

bors commented May 10, 2015

📌 Commit 2274bf1 has been approved by Manishearth

bors added a commit that referenced this pull request May 10, 2015
@bors
Copy link
Contributor

bors commented May 10, 2015

⌛ Testing commit 2274bf1 with merge 750f2c6...

@Manishearth
Copy link
Member Author

@bors: force

@bors bors merged commit 2274bf1 into rust-lang:master May 10, 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.

9 participants