summaryrefslogtreecommitdiffstats
path: root/src/test/ui/lint/for_loop_over_fallibles.rs
blob: 43d71c2e808a91d5c261aba452ad9808671f735f (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
// check-pass

fn main() {
    // Common
    for _ in Some(1) {}
    //~^ WARN for loop over an `Option`. This is more readably written as an `if let` statement
    //~| HELP to check pattern in a loop use `while let`
    //~| HELP consider using `if let` to clear intent
    for _ in Ok::<_, ()>(1) {}
    //~^ WARN for loop over a `Result`. This is more readably written as an `if let` statement
    //~| HELP to check pattern in a loop use `while let`
    //~| HELP consider using `if let` to clear intent

    // `Iterator::next` specific
    for _ in [0; 0].iter().next() {}
    //~^ WARN for loop over an `Option`. This is more readably written as an `if let` statement
    //~| HELP to iterate over `[0; 0].iter()` remove the call to `next`
    //~| HELP consider using `if let` to clear intent

    // `Result<impl Iterator, _>`, but function doesn't return `Result`
    for _ in Ok::<_, ()>([0; 0].iter()) {}
    //~^ WARN for loop over a `Result`. This is more readably written as an `if let` statement
    //~| HELP to check pattern in a loop use `while let`
    //~| HELP consider using `if let` to clear intent
}

fn _returns_result() -> Result<(), ()> {
    // `Result<impl Iterator, _>`
    for _ in Ok::<_, ()>([0; 0].iter()) {}
    //~^ WARN for loop over a `Result`. This is more readably written as an `if let` statement
    //~| HELP to check pattern in a loop use `while let`
    //~| HELP consider unwrapping the `Result` with `?` to iterate over its contents
    //~| HELP consider using `if let` to clear intent

    // `Result<impl IntoIterator>`
    for _ in Ok::<_, ()>([0; 0]) {}
    //~^ WARN for loop over a `Result`. This is more readably written as an `if let` statement
    //~| HELP to check pattern in a loop use `while let`
    //~| HELP consider unwrapping the `Result` with `?` to iterate over its contents
    //~| HELP consider using `if let` to clear intent

    Ok(())
}