diff options
Diffstat (limited to '')
-rw-r--r-- | src/tools/clippy/tests/ui/for_kv_map.rs | 50 |
1 files changed, 50 insertions, 0 deletions
diff --git a/src/tools/clippy/tests/ui/for_kv_map.rs b/src/tools/clippy/tests/ui/for_kv_map.rs new file mode 100644 index 000000000..39a8d960a --- /dev/null +++ b/src/tools/clippy/tests/ui/for_kv_map.rs @@ -0,0 +1,50 @@ +#![warn(clippy::for_kv_map)] +#![allow(clippy::used_underscore_binding)] + +use std::collections::*; +use std::rc::Rc; + +fn main() { + let m: HashMap<u64, u64> = HashMap::new(); + for (_, v) in &m { + let _v = v; + } + + let m: Rc<HashMap<u64, u64>> = Rc::new(HashMap::new()); + for (_, v) in &*m { + let _v = v; + // Here the `*` is not actually necessary, but the test tests that we don't + // suggest + // `in *m.values()` as we used to + } + + let mut m: HashMap<u64, u64> = HashMap::new(); + for (_, v) in &mut m { + let _v = v; + } + + let m: &mut HashMap<u64, u64> = &mut HashMap::new(); + for (_, v) in &mut *m { + let _v = v; + } + + let m: HashMap<u64, u64> = HashMap::new(); + let rm = &m; + for (k, _value) in rm { + let _k = k; + } + + // The following should not produce warnings. + + let m: HashMap<u64, u64> = HashMap::new(); + // No error, _value is actually used + for (k, _value) in &m { + let _ = _value; + let _k = k; + } + + let m: HashMap<u64, String> = Default::default(); + for (_, v) in m { + let _v = v; + } +} |