summaryrefslogtreecommitdiffstats
path: root/src/test/ui/lint/lint-unnecessary-parens.fixed
blob: 9c144324f2f7edc1112d228cad6309e3b063d904 (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
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
// run-rustfix

#![deny(unused_parens)]
#![allow(while_true)] // for rustfix

#[derive(Eq, PartialEq)]
struct X { y: bool }
impl X {
    fn foo(&self, conjunct: bool) -> bool { self.y && conjunct }
}

fn foo() -> isize {
    return 1; //~ ERROR unnecessary parentheses around `return` value
}
fn bar(y: bool) -> X {
    return X { y }; //~ ERROR unnecessary parentheses around `return` value
}

pub fn unused_parens_around_return_type() -> u32 { //~ ERROR unnecessary parentheses around type
    panic!()
}

pub fn unused_parens_around_block_return() -> u32 {
    let _foo = {
        5 //~ ERROR unnecessary parentheses around block return value
    };
    5 //~ ERROR unnecessary parentheses around block return value
}

pub trait Trait {
    fn test(&self);
}

pub fn passes_unused_parens_lint() -> &'static (dyn Trait) {
    panic!()
}

macro_rules! baz {
    ($($foo:expr),+) => {
        ($($foo),*)
    }
}

pub const CONST_ITEM: usize = 10; //~ ERROR unnecessary parentheses around assigned value
pub static STATIC_ITEM: usize = 10; //~ ERROR unnecessary parentheses around assigned value

fn main() {
    foo();
    bar(true); //~ ERROR unnecessary parentheses around function argument

    if true {} //~ ERROR unnecessary parentheses around `if` condition
    while true {} //~ ERROR unnecessary parentheses around `while` condition
    match true { //~ ERROR unnecessary parentheses around `match` scrutinee expression
        _ => {}
    }
    if let 1 = 1 {} //~ ERROR unnecessary parentheses around `let` scrutinee expression
    while let 1 = 2 {} //~ ERROR unnecessary parentheses around `let` scrutinee expression
    let v = X { y: false };
    // struct lits needs parens, so these shouldn't warn.
    if (v == X { y: true }) {}
    if (X { y: true } == v) {}
    if (X { y: false }.y) {}
    // this shouldn't warn, because the parens are necessary to disambiguate let chains
    if let true = (true && false) {}

    while (X { y: false }.foo(true)) {}
    while (true | X { y: false }.y) {}

    match (X { y: false }) {
        _ => {}
    }

    X { y: false }.foo(true); //~ ERROR unnecessary parentheses around method argument

    let mut _a = 0; //~ ERROR unnecessary parentheses around assigned value
    _a = 0; //~ ERROR unnecessary parentheses around assigned value
    _a += 1; //~ ERROR unnecessary parentheses around assigned value

    let _a = baz!(3, 4);
    let _b = baz!(3);
}