Skip to content

Use AtomicUsize::new(0) in place of deprecated ATOMIC_USIZE_INIT #1612

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

Merged
merged 1 commit into from
Feb 3, 2019

Conversation

jtgeibel
Copy link
Member

ATOMIC_USIZE_INIT is deprecated starting in 1.34. The new warning is
causing nightly to fail on CI. For consistency, Once::new() is used
in place of ONCE_INIT.

`ATOMIC_USIZE_INIT` is deprecated starting in 1.34.  The new warning is
causing nightly to fail on CI.  For consistency, `Once::new()` is used
in place of `ONCE_INIT`.
Copy link
Contributor

@locks locks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking clean 👌

@sgrif
Copy link
Contributor

sgrif commented Feb 3, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Feb 3, 2019

📌 Commit e54b8bb has been approved by sgrif

@bors
Copy link
Contributor

bors commented Feb 3, 2019

⌛ Testing commit e54b8bb with merge e783149...

bors added a commit that referenced this pull request Feb 3, 2019
Use `AtomicUsize::new(0)` in place of deprecated `ATOMIC_USIZE_INIT`

`ATOMIC_USIZE_INIT` is deprecated starting in 1.34.  The new warning is
causing nightly to fail on CI.  For consistency, `Once::new()` is used
in place of `ONCE_INIT`.
@sgrif
Copy link
Contributor

sgrif commented Feb 3, 2019

screen shot 2019-02-03 at 12 47 30 pm

Greetings from the future

@bors
Copy link
Contributor

bors commented Feb 3, 2019

☀️ Test successful - checks-travis
Approved by: sgrif
Pushing e783149 to master...

@bors bors merged commit e54b8bb into rust-lang:master Feb 3, 2019
@jtgeibel jtgeibel deleted the atomic-usize-new branch February 7, 2019 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants