Skip to content

Rollup of 7 pull requests #25352

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

Closed
wants to merge 34 commits into from
Closed

Conversation

bombless and others added 29 commits April 30, 2015 23:41
Attempted to organize them in a way more relevant to what newbies
would be interested in hearing.
This also updates the error message to remove mention of "managed
heap", which was removed in 8a91d33.
This error code has been removed.
The text in iterators.md wasn't wrong, but it read awkwardly to my ear.
I've just added a link instead to the lifetime elision section of
the book.
Attempted to organize them in a way more relevant to what newbies
would be interested in hearing.

I am not satisfied by this at all, but by virtue of deleting old links alone I think it is an improvement.

r? @steveklabnik
…xcrichton

This is a little rough, and it needs squashed and section headers, but i'd like to get some eyes on it sooner rather than later.
This makes the compatibility matrix render a little nicer on github, and also removes a note about windows support from 0.12 (Which is immaterial now that we're approaching 1.0)
Adds long diagnostic messages for the following error codes:

 - E0049
 - E0050
 - E0066
 - E0069
 - E0106
 - E0107
 - E0166
 - E0201
 - E0322

cc rust-lang#24407
As it is, save-analysis appears to return the span for the 'mut' in a declaration 'static mut identifier...' instead of the identifier. This minor change appears to fix the problem, by skipping the mut when it is present.
@steveklabnik
Copy link
Member Author

Manually merged in the three that didn't automatically merge.

@steveklabnik
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Collaborator

bors commented May 13, 2015

📌 Commit 2c01eac has been approved by steveklabnik

@steveklabnik
Copy link
Member Author

@bors: force

@steveklabnik
Copy link
Member Author

@bors: r-

@steveklabnik
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Collaborator

bors commented May 13, 2015

📌 Commit 758d568 has been approved by steveklabnik

@steveklabnik
Copy link
Member Author

@bors: force

@bors
Copy link
Collaborator

bors commented May 13, 2015

⌛ Testing commit 758d568 with merge bd5fb16...

@bors
Copy link
Collaborator

bors commented May 13, 2015

💔 Test failed - auto-mac-64-opt

@steveklabnik
Copy link
Member Author

@bors: retry

@steveklabnik
Copy link
Member Author

@bors: force

@bors
Copy link
Collaborator

bors commented May 13, 2015

⌛ Testing commit 758d568 with merge f855894...

@bors
Copy link
Collaborator

bors commented May 13, 2015

💔 Test failed - auto-mac-64-nopt-t

@steveklabnik
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Collaborator

bors commented May 13, 2015

📌 Commit 1d34f92 has been approved by steveklabnik

@steveklabnik
Copy link
Member Author

@bors: retry

@steveklabnik
Copy link
Member Author

@bors: force

@bors
Copy link
Collaborator

bors commented May 13, 2015

⌛ Testing commit 1d34f92 with merge 25b08b4...

@bors
Copy link
Collaborator

bors commented May 13, 2015

💔 Test failed - auto-mac-32-opt

@Centril Centril added the rollup A PR which is a rollup label Oct 24, 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.