Skip to content
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

correct minor typo and bumped atomic-polyfill #190

Merged
merged 2 commits into from Aug 16, 2022

Conversation

jqnatividad
Copy link
Contributor

@jqnatividad jqnatividad commented Aug 13, 2022

No description provided.

@jqnatividad jqnatividad changed the title correct minor typo correct minor typo and bumped atomic-polyfill Aug 13, 2022
@matklad
Copy link
Owner

matklad commented Aug 16, 2022

bors r+

@bors
Copy link
Contributor

bors bot commented Aug 16, 2022

Build succeeded:

@bors bors bot merged commit f4645cb into matklad:master Aug 16, 2022
@matklad
Copy link
Owner

matklad commented Aug 16, 2022

released as https://crates.io/crates/once_cell/1.13.1

@taiki-e
Copy link

taiki-e commented Sep 9, 2022

FYI, this was a breaking change for users who use the atomic-polyfill feature. (atomic-polyfill itself is not used in the once_cell public API, but the mechanism to implement a critical session implementation in the atomic-polyfill depends on versions of the atomic-polyfill and the critical-section (used in atomic-polyfill), so this can cause a compile error. I can provide reproduction if needed.)

That said, I think this breaking change is justified because atomic-polyfill 1.0 has mitigated/fixed several soundness issues.

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.

None yet

3 participants