Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Stabilize atomic_fetch_update #83615

Closed
wants to merge 1 commit into from

Conversation

CDirkx
Copy link
Contributor

@CDirkx CDirkx commented Mar 28, 2021

Stabilize feature atomic_fetch_update; the fetch_update method for AtomicBool and AtomicPtr.

fetch_update has already been stabilized for all the atomic integers (#71843). As far as I can see, the reason that AtomicBool and AtomicPtr weren't stabilized earlier is that they were simply forgotten.

Tracking issue: #78639

Stabilizes `fetch_update` for `AtomicBool` and `AtomicPtr`
@rust-highfive
Copy link
Collaborator

r? @Mark-Simulacrum

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 28, 2021
@m-ou-se m-ou-se added S-waiting-on-fcp Status: PR is in FCP and is awaiting for FCP to complete. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 31, 2021
@m-ou-se m-ou-se assigned m-ou-se and unassigned Mark-Simulacrum Mar 31, 2021
@CDirkx
Copy link
Contributor Author

CDirkx commented Apr 14, 2021

Closing as per #84085

@CDirkx CDirkx closed this Apr 14, 2021
@m-ou-se
Copy link
Member

m-ou-se commented Apr 14, 2021

Oh sorry, I should've merged this instead of creating #84085!

@CDirkx CDirkx deleted the atomic_fetch_update branch April 14, 2021 17:31
@CDirkx
Copy link
Contributor Author

CDirkx commented Apr 15, 2021

No problem 👍🏻

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
S-waiting-on-fcp Status: PR is in FCP and is awaiting for FCP to complete.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants