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 9 pull requests #45046

Merged
merged 19 commits into from
Oct 5, 2017
Merged

Rollup of 9 pull requests #45046

merged 19 commits into from
Oct 5, 2017

Conversation

vitiral and others added 19 commits September 29, 2017 17:44
Upgrade comment to doc comment.
Also fixes some stdlib links to the reference which have changed.
some dataflow-tracked borrow-data entry.

Fix rust-lang#44828

(The comment thread on the aforementioned issue discusses why its best
to just remove this assertion.)
…veklabnik

Corrected the CONTRIBUTING.md "External Dependencies" section

The "External Dependencies" section is a little outdated.
Please see following comments rust-lang#44567 (comment) rust-lang#44567 (comment) for rationale.
Add links to headers in README and CONTRIBUTING

this also adds dependencies to CONTRIBUTING

I'm just getting started building the rust compiler and noticed this information/ability was missing.

It was also missing the gdb dependency for running tests. I pulled the information out of `appveyor.yml` and recommended later than 7.1 because that is what [apt ships](https://packages.ubuntu.com/search?suite=trusty&keywords=gdb). Feel free to tell me something different!
fix ItemKind::DefaultImpl doc comment

Upgrade comment to doc comment.

...Is this actually used? If so, why does the `Impl` variant right below have a `Defaultness`?
update books for next release

Also fixes some stdlib links to the reference which have changed.

First step in https://forge.rust-lang.org/release-process.html, doing it a few days early 😄
…exes-ice, r=arielb1

`EndRegion` do not always correspond to borrow-data entries

Remove assertion that the argument to every `EndRegion` correspond to some dataflow-tracked borrow-data entry.

Fix rust-lang#44828

(The comment thread on the aforementioned issue discusses why its best to just remove this assertion.)
Use HTTPS protocol for "chat.mibbit.com"

I changed the `http://` protocol to `https://` for the `chat.mibbit.com` website. 📝

--[**Suriyaa**](https://mozillians.org/de/u/suriyaakudo/) 🦊

(*PS: Is somebody interested to vouch me at https://mozillians.org/de/u/suriyaakudo/?*)
Add missing urls for Mutex

r? @rust-lang/docs
…m, r=pnkfelix

add the issue number to doc_masked's feature gate

Whoops, missed this in the original `#[doc(masked)]` PR.
@kennytm
Copy link
Member Author

kennytm commented Oct 5, 2017

@bors r+ p=9

@bors
Copy link
Contributor

bors commented Oct 5, 2017

📌 Commit 5440733 has been approved by kennytm

@bors
Copy link
Contributor

bors commented Oct 5, 2017

⌛ Testing commit 5440733 with merge 1db1144...

bors added a commit that referenced this pull request Oct 5, 2017
Rollup of 9 pull requests

- Successful merges: #44664, #44935, #44972, #44980, #44987, #44997, #45006, #45017, #45024
- Failed merges:
@bors
Copy link
Contributor

bors commented Oct 5, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: kennytm
Pushing 1db1144 to master...

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.