squiid/rust
1
0
Fork 0
Code Issues Pull requests Projects Activity

Auto merge of #3566 - kinnison:kinnison/typofix, r=phansch

Browse source 
mutex_atomic: Correct location of AtomicBool and friends

The AtomicBool, AtomicUsize, and friends, types live in the
`std::sync::atomic` module, rather than `std::atomic` as the lint
help text used to say.
This commit is contained in:
bors 2018-12-19 20:54:48 +00:00
commit f067c5c1ec
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
clippy_lints/src/mutex_atomic.rs
View file

@ -22,7 +22,7 @@ use crate::utils::{match_type, paths, span_lint};
///
/// **Why is this bad?** Using a mutex just to make access to a plain bool or
/// reference sequential is shooting flies with cannons.
/// `std::atomic::AtomicBool` and `std::atomic::AtomicPtr` are leaner and
/// `std::sync::atomic::AtomicBool` and `std::sync::atomic::AtomicPtr` are leaner and
/// faster.
///
/// **Known problems:** This lint cannot detect if the mutex is actually used
@ -43,7 +43,7 @@ declare_clippy_lint! {
///
/// **Why is this bad?** Using a mutex just to make access to a plain integer
/// sequential is
/// shooting flies with cannons. `std::atomic::usize` is leaner and faster.
/// shooting flies with cannons. `std::sync::atomic::AtomicUsize` is leaner and faster.
///
/// **Known problems:** This lint cannot detect if the mutex is actually used
/// for waiting before a critical section.