-
Notifications
You must be signed in to change notification settings - Fork 13k
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 #97304
Rollup of 5 pull requests #97304
Conversation
While already being inferable from the doc examples, it wasn't fully specified. This is the only logical way to do a slice iterator.
…rder, r=dtolnay Clarify slice and Vec iteration order While already being inferable from the doc examples, it wasn't fully specified. This is the only logical way to do a slice iterator, so I think this should be uncontroversial. It also improves the `Vec::into_iter` example to better show the order and that the iterator returns owned values.
Remove feature: `crate` visibility modifier FCP completed in rust-lang#53120.
…rrors Add regression test for rust-lang#91949 Closes rust-lang#91949 This needs `build-fail` because the original bug only appeared with `cargo build`. r? `@compiler-errors`
std::time : fix variable name in the doc
Fix some typos in arg checking algorithm Fixes rust-lang#97197 Also fixes a typo where if we're missing args A, B, C, we actually say A, B, B
@bors r+ rollup=never p=5 |
📌 Commit b5ff4ad has been approved by |
☀️ Test successful - checks-actions |
Finished benchmarking commit (9e2f655): comparison url. Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)Results
CyclesResults
If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf. @rustbot label: -perf-regression Footnotes |
Successful merges:
crate
visibility modifier #97254 (Remove feature:crate
visibility modifier)overflow
error. #91949)Failed merges:
r? @ghost
@rustbot modify labels: rollup
Create a similar rollup