about summary refs log tree commit diff
path: root/src/tools/clippy/tests/ui/missing_spin_loop.fixed
diff options
context:
space:
mode:
authorbors <bors@rust-lang.org>2022-03-14 16:24:12 +0000
committerbors <bors@rust-lang.org>2022-03-14 16:24:12 +0000
commit285fa7ecd05dcbfdaf2faaf20400f5f92b39b3c6 (patch)
treea8b13585116a678e07763e47514130459f5e4347 /src/tools/clippy/tests/ui/missing_spin_loop.fixed
parentbce19cf7f19ee5729defaccc86b068cc3c206c9c (diff)
parent706fa54456f42086a943992e03354c75e1754307 (diff)
downloadrust-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.fixed28
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
+}