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 6 pull requests #29444

Merged
merged 19 commits into from Oct 29, 2015
Merged

Rollup of 6 pull requests #29444

merged 19 commits into from Oct 29, 2015

Conversation

mdinger and others added 19 commits October 23, 2015 21:07
…m message

I recently discovered that this is not mentioned in the docs, only in
the examples, and it's not evident for people coming from C++

r? @steveklabnik
`as_local_node_id`, instead just compare against `Some(id)`.
Fixes rust-lang#29161.
Fix corner case in privacy that was causing ICEs when the `source_did` was not crate-local.

Full confession: I only kinda sorta understand this code, but afaict it's legit for `source_did` to be from another crate.

r? @alexcrichton
This is an alternative to rust-lang#29240 which fixes rust-lang#15307 by adding colors to primitives and aliases instead of underlining.

Try to keep the discussion in rust-lang#29240 for now though so it can be kept track of.

A sample rendering is [here](http://mdinger.github.io/rust_std_colored/std/index.html)
…m message

I recently discovered that this is not mentioned in the docs, only in
the examples, and it's not evident for people coming from C++

r? @steveklabnik
…=Manishearth

I put the reference under the function return operator `->` rather than near the suggested `!` operators as I thought it was more relevant there.

Resolves rust-lang#29431
…xcrichton

The raw byte string literal syntax in the syntax index was incorrect. Also added links to the reference for raw and/or byte string literals.
@steveklabnik
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Contributor

bors commented Oct 29, 2015

📌 Commit 0809eaa has been approved by steveklabnik

@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

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

@bors
Copy link
Contributor

bors commented Oct 29, 2015

⌛ Testing commit 0809eaa with merge b4af35f...

bors added a commit that referenced this pull request Oct 29, 2015
@bors bors merged commit 0809eaa into rust-lang:master Oct 29, 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

8 participants