Skip to content

Remove 64 bit atomics in main crate#8037

Open
kay-lambdadelta wants to merge 1 commit intoemilk:mainfrom
kay-lambdadelta:remove-64-bit-atomics
Open

Remove 64 bit atomics in main crate#8037
kay-lambdadelta wants to merge 1 commit intoemilk:mainfrom
kay-lambdadelta:remove-64-bit-atomics

Conversation

@kay-lambdadelta
Copy link
Copy Markdown

This allows the base egui crate to run on platforms without 64 bit atomics

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 27, 2026

Preview available at https://egui-pr-preview.github.io/pr/8037-remove-64-bit-atomics
Note that it might take a couple seconds for the update to show up after the preview_build workflow has completed.

View snapshot changes at kitdiff

Copy link
Copy Markdown
Collaborator

@lucasmerlin lucasmerlin left a comment

Choose a reason for hiding this comment

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

Thanks!
Maybe we can also forbid the use of AtomicU64 with clippy so we don't regress on this in the future?

@kay-lambdadelta
Copy link
Copy Markdown
Author

Thanks! Maybe we can also forbid the use of AtomicU64 with clippy so we don't regress on this in the future?

Something this would work I believe.

disallowed-types = [
    { path = "std::sync::atomic::AtomicU64", reason = "AtomicU64 is not supported on many common platforms, use AtomicU32 or AtomicUsize instead" }
]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants