summaryrefslogtreecommitdiffstats
path: root/src/tools/clippy/tests/ui/iter_kv_map.fixed
blob: 83fee04080fa76507d15d19f1376f35ed48caf96 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
// run-rustfix

#![warn(clippy::iter_kv_map)]
#![allow(clippy::redundant_clone)]
#![allow(clippy::suspicious_map)]
#![allow(clippy::map_identity)]

use std::collections::{BTreeMap, HashMap};

fn main() {
    let get_key = |(key, _val)| key;

    let map: HashMap<u32, u32> = HashMap::new();

    let _ = map.keys().collect::<Vec<_>>();
    let _ = map.values().collect::<Vec<_>>();
    let _ = map.values().map(|v| v + 2).collect::<Vec<_>>();

    let _ = map.clone().into_keys().collect::<Vec<_>>();
    let _ = map.clone().into_keys().map(|key| key + 2).collect::<Vec<_>>();

    let _ = map.clone().into_values().collect::<Vec<_>>();
    let _ = map.clone().into_values().map(|val| val + 2).collect::<Vec<_>>();

    let _ = map.clone().values().collect::<Vec<_>>();
    let _ = map.keys().filter(|x| *x % 2 == 0).count();

    // Don't lint
    let _ = map.iter().filter(|(_, val)| *val % 2 == 0).map(|(key, _)| key).count();
    let _ = map.iter().map(get_key).collect::<Vec<_>>();

    // Linting the following could be an improvement to the lint
    // map.iter().filter_map(|(_, val)| (val % 2 == 0).then(val * 17)).count();

    // Lint
    let _ = map.keys().map(|key| key * 9).count();
    let _ = map.values().map(|value| value * 17).count();

    let map: BTreeMap<u32, u32> = BTreeMap::new();

    let _ = map.keys().collect::<Vec<_>>();
    let _ = map.values().collect::<Vec<_>>();
    let _ = map.values().map(|v| v + 2).collect::<Vec<_>>();

    let _ = map.clone().into_keys().collect::<Vec<_>>();
    let _ = map.clone().into_keys().map(|key| key + 2).collect::<Vec<_>>();

    let _ = map.clone().into_values().collect::<Vec<_>>();
    let _ = map.clone().into_values().map(|val| val + 2).collect::<Vec<_>>();

    let _ = map.clone().values().collect::<Vec<_>>();
    let _ = map.keys().filter(|x| *x % 2 == 0).count();

    // Don't lint
    let _ = map.iter().filter(|(_, val)| *val % 2 == 0).map(|(key, _)| key).count();
    let _ = map.iter().map(get_key).collect::<Vec<_>>();

    // Linting the following could be an improvement to the lint
    // map.iter().filter_map(|(_, val)| (val % 2 == 0).then(val * 17)).count();

    // Lint
    let _ = map.keys().map(|key| key * 9).count();
    let _ = map.values().map(|value| value * 17).count();
}