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 5 pull requests #40899

Merged
merged 15 commits into from
Mar 29, 2017
Merged

Rollup of 5 pull requests #40899

merged 15 commits into from
Mar 29, 2017

Conversation

mitsuhiko and others added 15 commits March 22, 2017 00:01
This converts all of borrowck's `mut` suggestions to a new
`mc::ImmutabilityBlame` API instead of the current mix of various hacks.

Fixes rust-lang#35937.
Fixes rust-lang#40823.
Add all unstable features to the Unstable Book, also remove a few that
either no longer exist or were promoted to stable.

These changes were extracted out of
rust-lang#40694
…ushi

Added core::cmp::Reverse for sort_by_key reverse sorting

I'm not sure if this is the best way to go about proposing this feature but it's pretty useful. It allows you to use `sort_by_key` and return tuples where a single item is then reversed to how it normally sorts.

I quite miss something like this in Rust currently though I'm not sure if this is the best way to implement it.
…atures, r=steveklabnik

Add all unstable features to Unstable Book.

Add all unstable features to the Unstable Book, also remove a few that
either no longer exist or were promoted to stable.

These changes were extracted out of
rust-lang#40694
borrowck: consolidate `mut` suggestions

This converts all of borrowck's `mut` suggestions to a new
`mc::ImmutabilityBlame` API instead of the current mix of various hacks.

Fixes rust-lang#35937.
Fixes rust-lang#40823.
Fixes rust-lang#40859.

cc @estebank
r? @pnkfelix
Adding linking for Once docs rust-lang#29377

Linking everything around `Once`, `ONCE_INIT`, and `OnceState`.
@frewsxcv
Copy link
Member Author

@bors r+ p=10

@bors
Copy link
Contributor

bors commented Mar 29, 2017

📌 Commit 8df6cd3 has been approved by frewsxcv

@rust-highfive
Copy link
Collaborator

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @GuillaumeGomez (or someone else) soon.

If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.

Please see the contribution instructions for more information.

@bors
Copy link
Contributor

bors commented Mar 29, 2017

⌛ Testing commit 8df6cd3 with merge cd5502d...

@bors
Copy link
Contributor

bors commented Mar 29, 2017

💔 Test failed - status-appveyor

@frewsxcv
Copy link
Member Author

In file included from C:\projects\rust\src\llvm\lib\Target\SystemZ\AsmParser\SystemZAsmParser.cpp:491:0:
lib/Target/SystemZ/SystemZGenAsmMatcher.inc:3156:6: warning: 'bool {anonymous}::SystemZAsmParser::mnemonicIsValid(llvm::StringRef, unsigned int)' defined but not used [-Wunused-function]
 bool SystemZAsmParser::
      ^~~~~~~~~~~~~~~~
[1242/1629] Building CXX object lib/Target/SystemZ/CMakeFiles/LLVMSystemZCodeGen.dir/SystemZTDC.cpp.obj
DEBUG:sccache::commands: Server sent CompileStarted
[1243/1629] Linking CXX static library lib\libLLVMSystemZAsmParser.a
FAILED: lib/libLLVMSystemZAsmParser.a 
cmd.exe /C "cd . && "C:\Program Files (x86)\CMake\bin\cmake.exe" -E remove lib\libLLVMSystemZAsmParser.a && C:\projects\rust\mingw32\bin\ar.exe qc lib\libLLVMSystemZAsmParser.a  lib/Target/SystemZ/AsmParser/CMakeFiles/LLVMSystemZAsmParser.dir/SystemZAsmParser.cpp.obj && C:\projects\rust\mingw32\bin\ranlib.exe lib\libLLVMSystemZAsmParser.a && cd ."
C:\projects\rust\mingw32\bin\ranlib.exe: unable to rename 'lib\libLLVMSystemZAsmParser.a'; reason: Permission denied
[1244/1629] Linking CXX static library lib\libLLVMSystemZCodeGen.a
FAILED: lib/libLLVMSystemZCodeGen.a 

Seems spurious and unrelated?

@bors retry

@bors
Copy link
Contributor

bors commented Mar 29, 2017

⌛ Testing commit 8df6cd3 with merge e1cec5d...

bors added a commit that referenced this pull request Mar 29, 2017
Rollup of 5 pull requests

- Successful merges: #40720, #40786, #40841, #40866, #40897
- Failed merges:
@bors
Copy link
Contributor

bors commented Mar 29, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: frewsxcv
Pushing e1cec5d to master...

@bors bors merged commit 8df6cd3 into rust-lang:master Mar 29, 2017
@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.

9 participants