Stabilize atomic_try_updateand deprecate fetch_update starting 1.99.0#148590
Stabilize atomic_try_updateand deprecate fetch_update starting 1.99.0#148590rust-bors[bot] merged 1 commit intorust-lang:mainfrom
atomic_try_updateand deprecate fetch_update starting 1.99.0#148590Conversation
|
I would suggest splitting out the lint change into a separate PR. It would be useful even before stabilization, I don't think it requires its own FCP, and it'll make the API stabilization itself a bit easier to review. That's especially important since the simultaneous deprecation makes it a big and tricky diff by stabilization standards! |
02e586c to
67b2e92
Compare
|
Good idea! Done: #148601 |
…ng, r=Kivooeo `invalid_atomic_ordering`: also lint `update` & `try_update` Split from rust-lang#148590 Tracking issue for `update` and `try_update`: rust-lang#135894
…ng, r=Kivooeo `invalid_atomic_ordering`: also lint `update` & `try_update` Split from rust-lang#148590 Tracking issue for `update` and `try_update`: rust-lang#135894
…ng, r=Kivooeo `invalid_atomic_ordering`: also lint `update` & `try_update` Split from rust-lang#148590 Tracking issue for `update` and `try_update`: rust-lang#135894
67b2e92 to
7ffe34e
Compare
…ooeo `invalid_atomic_ordering`: also lint `update` & `try_update` Split from rust-lang/rust#148590 Tracking issue for `update` and `try_update`: rust-lang/rust#135894
…ooeo `invalid_atomic_ordering`: also lint `update` & `try_update` Split from rust-lang/rust#148590 Tracking issue for `update` and `try_update`: rust-lang/rust#135894
…ng, r=Kivooeo `invalid_atomic_ordering`: also lint `update` & `try_update` Split from rust-lang#148590 Tracking issue for `update` and `try_update`: rust-lang#135894
…ooeo `invalid_atomic_ordering`: also lint `update` & `try_update` Split from rust-lang/rust#148590 Tracking issue for `update` and `try_update`: rust-lang/rust#135894
|
☔ The latest upstream changes (presumably #149222) made this pull request unmergeable. Please resolve the merge conflicts. |
7ffe34e to
9a0eb78
Compare
This comment has been minimized.
This comment has been minimized.
9a0eb78 to
0733988
Compare
This comment has been minimized.
This comment has been minimized.
|
@rustbot label -S-waiting-on-FCP RIP |
0733988 to
65e453c
Compare
|
@rustbot reroll Or what was that new command named |
library/core/src/sync/atomic.rs
Outdated
| #[cfg_attr(miri, track_caller)] // even without panics, this helps for Miri backtraces | ||
| #[rustc_should_not_be_called_on_const_items] | ||
| #[deprecated( | ||
| since = "1.96.0", |
There was a problem hiding this comment.
ping @Amanieu from #135894 (comment)
Point of clarification. Is it intended to deprecate this immediately with stabilization, have this set to a version further down the line, or simply TBD like the u8 module?
(also, does CURRENT_RUSTC_VERSION work here?)
There was a problem hiding this comment.
No, it's fine to delay the deprecation by a few versions. This will give people time to transition to the new API without warnings.
There was a problem hiding this comment.
@GrigorenkoPV Let's change it to 1.99 then. That'll be an 18 week period between stabilization and deprecation.
and deprecate fetch_update starting 1.99.0
65e453c to
3a48b9f
Compare
|
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
atomic_try_updateand deprecate fetch_update starting 1.96.0atomic_try_updateand deprecate fetch_update starting 1.99.0
|
@bors r+ |
…jhpratt Stabilize `atomic_try_update`and deprecate `fetch_update` starting 1.99.0 Tracking issue: rust-lang#135894 FCP completed: rust-lang#135894 (comment) ~1.96.0 was chosen because I don't think the remaining month until 1.93.0 becomes beta is enough for the FCP to finish and this to get merged, so 1.94.0 + a couple of versions of leeway: rust-lang#135894 (comment) 1.99 suggested in rust-lang#148590 (comment) Closes rust-lang#135894
…jhpratt Stabilize `atomic_try_update`and deprecate `fetch_update` starting 1.99.0 Tracking issue: rust-lang#135894 FCP completed: rust-lang#135894 (comment) ~1.96.0 was chosen because I don't think the remaining month until 1.93.0 becomes beta is enough for the FCP to finish and this to get merged, so 1.94.0 + a couple of versions of leeway: rust-lang#135894 (comment) 1.99 suggested in rust-lang#148590 (comment) Closes rust-lang#135894
|
⌛ Testing commit 3a48b9f with merge 83d71ef... Workflow: https://github.com/rust-lang/rust/actions/runs/21779822668 |
Stabilize `atomic_try_update`and deprecate `fetch_update` starting 1.99.0 Tracking issue: #135894 FCP completed: #135894 (comment) ~1.96.0 was chosen because I don't think the remaining month until 1.93.0 becomes beta is enough for the FCP to finish and this to get merged, so 1.94.0 + a couple of versions of leeway: #135894 (comment) 1.99 suggested in #148590 (comment) Closes #135894
|
@bors yield |
|
Auto build cancelled. Cancelled workflows: The next pull request likely to be tested is #152282. |
…uwer Rollup of 8 pull requests Successful merges: - #148590 (Stabilize `atomic_try_update`and deprecate `fetch_update` starting 1.99.0) - #150522 (Stabilize new inclusive range type and iterator type) - #152235 (Convert to inline diagnostics in `rustc_parse`) - #152267 (feat: Implement `int_from_ascii` for `NonZero<T>`) - #151576 (Stabilize `core::hint::cold_path`) - #151933 (Linker-plugin-based LTO: give an explanation how to use linker-plugin-lto with full LTO) - #152010 (Ignore all debuginfo tests for LLDB that we do not run in CI) - #152199 (Move `rustc_query_system::cache`.)
Rollup merge of #148590 - GrigorenkoPV:atomic_try_update, r=jhpratt Stabilize `atomic_try_update`and deprecate `fetch_update` starting 1.99.0 Tracking issue: #135894 FCP completed: #135894 (comment) ~1.96.0 was chosen because I don't think the remaining month until 1.93.0 becomes beta is enough for the FCP to finish and this to get merged, so 1.94.0 + a couple of versions of leeway: #135894 (comment) 1.99 suggested in #148590 (comment) Closes #135894
Tracking issue: #135894
FCP completed: #135894 (comment)
1.96.0 was chosen because I don't think the remaining month until 1.93.0 becomes beta is enough for the FCP to finish and this to get merged, so 1.94.0 + a couple of versions of leeway: #135894 (comment)1.99 suggested in #148590 (comment)
Closes #135894