Skip to content

Rollup of 9 pull requests #32882

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

Merged
merged 20 commits into from
Apr 12, 2016
Merged

Rollup of 9 pull requests #32882

merged 20 commits into from
Apr 12, 2016

Conversation

nikomatsakis and others added 20 commits April 7, 2016 15:15
The asm generated for b < 128 || b >= 192 is not ideal, as it computes
both sub-inequalities. This patch replaces it with bit magic.

Fixes rust-lang#32471
* Copy documentation from signed implementation to unsigned implementation, where necessary.
* Use signed integers in signed documentation, where possible.
Adds data race in docs

Thanks for all your hard work!
This is in reference to rust-lang#32733
I know there has been a discussion about this on PR rust-lang#32538 so you are welcome to keep the code as is or merge my documentation in.
Let me know what you think and/or if you want me to modify anything!
Read "is-like-msvc" target option from JSON

This is in reference to rust-lang#32818.
…tics, r=eddyb

resolve: import resolution diagnostics

This improves the diagnostics for failing import resolutions (fixes rust-lang#32833).
r? @eddyb
…labnik

Add some missing commas and missing titles/formatting

Fixes rust-lang#29373.

r? @steveklabnik
Bit-magic for faster is_char_boundary

The asm generated for b < 128 || b >= 192 is not ideal, as it computes
both sub-inequalities. This patch replaces it with bit magic.

Fixes rust-lang#32471
Match signed/unsigned integer type docs

* Copy documentation from signed implementation to unsigned implementation, where necessary.

A few functions had elaborate docs in the signed version but not in the unsigned version. This probably happenned because the signed version is at the top and the author didn't realize they had to update the documentation in both locations.

* Use signed integers in signed documentation, where possible.

r? @steveklabnik
@steveklabnik
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Collaborator

bors commented Apr 11, 2016

📌 Commit 55e90bb has been approved by steveklabnik

@rust-highfive
Copy link
Contributor

r? @alexcrichton

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

@bors
Copy link
Collaborator

bors commented Apr 11, 2016

⌛ Testing commit 55e90bb with merge a68a291...

@alexcrichton
Copy link
Member

@bors: retry force clean

@bors
Copy link
Collaborator

bors commented Apr 11, 2016

⌛ Testing commit 55e90bb with merge 081fdc1...

@brson
Copy link
Contributor

brson commented Apr 11, 2016

@bors: retry force clean

@bors
Copy link
Collaborator

bors commented Apr 11, 2016

⌛ Testing commit 55e90bb with merge c0221c8...

bors added a commit that referenced this pull request Apr 11, 2016
Rollup of 9 pull requests

- Successful merges: #32768, #32802, #32815, #32823, #32849, #32854, #32862, #32870, #32873
- Failed merges:
@bors
Copy link
Collaborator

bors commented Apr 11, 2016

💔 Test failed - auto-win-msvc-32-opt

@alexcrichton
Copy link
Member

@bors: retry

On Mon, Apr 11, 2016 at 3:17 PM, bors [email protected] wrote:

[image: 💔] Test failed - auto-win-msvc-32-opt
http://buildbot.rust-lang.org/builders/auto-win-msvc-32-opt/builds/2941


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#32882 (comment)

@bors bors merged commit 55e90bb into rust-lang:master Apr 12, 2016
@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.