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 other PRs in the queue; Friday #24759

Merged
merged 27 commits into from
Apr 24, 2015
Merged

Conversation

Manishearth
Copy link
Member

rapha and others added 26 commits April 24, 2015 02:43
Consistency. The book also refers to it as trait objects.
'win32' -> 'windows', added 'ios'
* Add two missing keywords, `Self` and `proc`
* Fix some mis-alphabetized keywords
This was merged back into the regular macros chapter, but a few
references lingered and caused broken links.
1) Moved 'while' section below 'loop', 'break', and 'continue'; 2) Added information to 'while' and 'for' loops that they interact with 'break' and 'continue' and may have a lifetime label. 3) Clarified labeling syntax on the infinite loops.
Updated sample code to updated syntax (now compiles). Also tweaked the text to reflect the change.
Add section for range expressions.
 Consistency. The book also refers to it as trait objects.
 (And other kinds of items).

This brings trait/impl items in line with top-level items in this respect. It means we don't get the attributes when showing error messages for methods, etc. (a good thing, I think). Since attributes have spans it is still easy to create a span for the item + all attributes.
 * Add two missing keywords, `Self` and `proc`
* Fix some mis-alphabetized keywords

r? @steveklabnik
 1) Moved 'while' section below 'loop', 'break', and 'continue'; 2) Added information to 'while' and 'for' loops that they interact with 'break' and 'continue' and may have a lifetime label. 3) Clarified labeling syntax on the infinite loops.
 Updated sample code to updated syntax (now compiles). Also tweaked the text to reflect the change.
@Manishearth
Copy link
Member Author

@bors: r+ p=20

@bors
Copy link
Contributor

bors commented Apr 24, 2015

📌 Commit a33b5d3 has been approved by Manishearth

@Manishearth
Copy link
Member Author

@bors: r+ p=20

@bors
Copy link
Contributor

bors commented Apr 24, 2015

📌 Commit 131b5ac has been approved by Manishearth

@bors
Copy link
Contributor

bors commented Apr 24, 2015

⌛ Testing commit 131b5ac with merge 9d439b4...

@bors bors merged commit 131b5ac into rust-lang:master Apr 24, 2015
@Manishearth Manishearth deleted the rollup2 branch September 3, 2015 17:46
@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.