diff options
| author | bors <bors@rust-lang.org> | 2022-03-14 16:24:12 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2022-03-14 16:24:12 +0000 |
| commit | 285fa7ecd05dcbfdaf2faaf20400f5f92b39b3c6 (patch) | |
| tree | a8b13585116a678e07763e47514130459f5e4347 /src/tools/clippy/tests/ui/missing_spin_loop.fixed | |
| parent | bce19cf7f19ee5729defaccc86b068cc3c206c9c (diff) | |
| parent | 706fa54456f42086a943992e03354c75e1754307 (diff) | |
| download | rust-285fa7ecd05dcbfdaf2faaf20400f5f92b39b3c6.tar.gz rust-285fa7ecd05dcbfdaf2faaf20400f5f92b39b3c6.zip | |
Auto merge of #94929 - flip1995:clippyup, r=Manishearth
Update Clippy r? `@Manishearth` A few days delayed, because I recovered from a cold last week and couldn't get myself to do the sync, sorry. :upside_down_face:
Diffstat (limited to 'src/tools/clippy/tests/ui/missing_spin_loop.fixed')
| -rw-r--r-- | src/tools/clippy/tests/ui/missing_spin_loop.fixed | 28 |
1 files changed, 28 insertions, 0 deletions
diff --git a/src/tools/clippy/tests/ui/missing_spin_loop.fixed b/src/tools/clippy/tests/ui/missing_spin_loop.fixed new file mode 100644 index 00000000000..aa89e04d26e --- /dev/null +++ b/src/tools/clippy/tests/ui/missing_spin_loop.fixed @@ -0,0 +1,28 @@ +// run-rustfix +#![warn(clippy::missing_spin_loop)] +#![allow(clippy::bool_comparison)] +#![allow(unused_braces)] + +use core::sync::atomic::{AtomicBool, Ordering}; + +fn main() { + let b = AtomicBool::new(true); + // Those should lint + while b.load(Ordering::Acquire) { std::hint::spin_loop() } + + while !b.load(Ordering::SeqCst) { std::hint::spin_loop() } + + while b.load(Ordering::Acquire) == false { std::hint::spin_loop() } + + while { true == b.load(Ordering::Acquire) } { std::hint::spin_loop() } + + while b.compare_exchange(true, false, Ordering::Acquire, Ordering::Relaxed) != Ok(true) { std::hint::spin_loop() } + + while Ok(false) != b.compare_exchange(false, true, Ordering::Acquire, Ordering::Relaxed) { std::hint::spin_loop() } + + // This is OK, as the body is not empty + while b.load(Ordering::Acquire) { + std::hint::spin_loop() + } + // TODO: also match on loop+match or while let +} |
