-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
Remove deprecated modes #3308
Merged
Merged
Remove deprecated modes #3308
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Just added cmp.rs to the mix. This commit may require changes to the following Servo files:
|
This commit may require changes to the following Servo files: * rust-geom/matrix.rs * servo/platform/osmain.rs * rust-layers/layers.rs * rust-geom/matrix.rs
Now rebased on top of incoming (340a219) and passing |
Thanks! |
matthiaskrgr
added a commit
to matthiaskrgr/rust
that referenced
this pull request
Oct 15, 2018
Changes: ```` fix for rustc master mem_forget: fix syntax error in code sample explicit_counter_loop fix rust-lang#3308 false positive new_ret_no_self test remove tool lints cfg flag Added new_ret_no_self exception to clippy to pass dogfood tests Removed new_ret_no_self tests from method.rs new_ret_no_self correctly lint impl return Removed unused variables new_ret_no_self fix false positive for impl trait return with associated type self new_ret_no_self corrected panic and added test stderr new_ret_no_self added positive test cases Fix some more `stutter` warnings unused unit lint Install Windows SDK 10.0 on travis cmp_owned refactor cmp_owned correct error message if rhs is deref Add a comment reminding to update README if the default changes Specify which categories are enabled by default Only run markdown linter on linux Move Travis Windows build to allowed failures Add Travis windows build Fix `doc_markdown` lints Fix `stutter` lints Fix `similar_names` warnings cmp_owned current suggestion for multiple deref cmp_owned add test for multiple dereference Corrected single-character string constant used as pattern found in dogfood test Fixes 3289, cmp_owned wording and false positive ````
bors
added a commit
that referenced
this pull request
Oct 15, 2018
submodules: update clippy from 9d33731 to 5afdf8b Should fix toolstate. Changes: ```` fix for rustc master mem_forget: fix syntax error in code sample explicit_counter_loop fix #3308 false positive new_ret_no_self test remove tool lints cfg flag Added new_ret_no_self exception to clippy to pass dogfood tests Removed new_ret_no_self tests from method.rs new_ret_no_self correctly lint impl return Removed unused variables new_ret_no_self fix false positive for impl trait return with associated type self new_ret_no_self corrected panic and added test stderr new_ret_no_self added positive test cases Fix some more `stutter` warnings unused unit lint Install Windows SDK 10.0 on travis cmp_owned refactor cmp_owned correct error message if rhs is deref Add a comment reminding to update README if the default changes Specify which categories are enabled by default Only run markdown linter on linux Move Travis Windows build to allowed failures Add Travis windows build Fix `doc_markdown` lints Fix `stutter` lints Fix `similar_names` warnings cmp_owned current suggestion for multiple deref cmp_owned add test for multiple dereference Corrected single-character string constant used as pattern found in dogfood test Fixes 3289, cmp_owned wording and false positive ````
bors
pushed a commit
to rust-lang-ci/rust
that referenced
this pull request
May 15, 2021
Prioritize single_line_fn and empty_item_single_line over brace_style
RalfJung
pushed a commit
to RalfJung/rust
that referenced
this pull request
Feb 25, 2024
Test windows random shims This adds tests for `BCryptGenRandom` and `RtlGenRandom` (aka `SystemFunction036`). Note that `BCryptGenRandom` comes in two flavours: ```rust BCryptGenRandom(null_mut(), key.as_mut_ptr(), len, BCRYPT_USE_SYSTEM_PREFERRED_RNG) BCryptGenRandom(BCRYPT_RNG_ALG_HANDLE, key.as_mut_ptr(), len, 0) ``` Fixes rust-lang#3308
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
These commits remove deprecated modes from cmp.rs, dbg.rs, rope.rs, and list.rs (and delete list::push).
I haven't yet managed to catchincoming
when it was problem free, so this hasn't been all the way through amake check
. Let me know whenincoming
is clean...