Skip to content

Commit

Permalink
Add run-rustfix for wildcard_enum_match_arm lint
Browse files Browse the repository at this point in the history
  • Loading branch information
phansch committed Sep 21, 2019
1 parent f21cd81 commit afd7b18
Show file tree
Hide file tree
Showing 3 changed files with 73 additions and 5 deletions.
65 changes: 65 additions & 0 deletions tests/ui/wildcard_enum_match_arm.fixed
@@ -0,0 +1,65 @@
// run-rustfix

#![deny(clippy::wildcard_enum_match_arm)]
#![allow(unreachable_code, unused_variables)]

#[derive(Clone, Copy, Debug, Eq, PartialEq)]
enum Color {
Red,
Green,
Blue,
Rgb(u8, u8, u8),
Cyan,
}

impl Color {
fn is_monochrome(self) -> bool {
match self {
Color::Red | Color::Green | Color::Blue => true,
Color::Rgb(r, g, b) => r | g == 0 || r | b == 0 || g | b == 0,
Color::Cyan => false,
}
}
}

fn main() {
let color = Color::Rgb(0, 0, 127);
match color {
Color::Red => println!("Red"),
Color::Green | Color::Blue | Color::Rgb(..) | Color::Cyan => eprintln!("Not red"),
};
match color {
Color::Red => println!("Red"),
_not_red @ Color::Green | _not_red @ Color::Blue | _not_red @ Color::Rgb(..) | _not_red @ Color::Cyan => eprintln!("Not red"),
};
let _str = match color {
Color::Red => "Red".to_owned(),
not_red @ Color::Green | not_red @ Color::Blue | not_red @ Color::Rgb(..) | not_red @ Color::Cyan => format!("{:?}", not_red),
};
match color {
Color::Red => {},
Color::Green => {},
Color::Blue => {},
Color::Cyan => {},
c if c.is_monochrome() => {},
Color::Rgb(_, _, _) => {},
};
let _str = match color {
Color::Red => "Red",
c @ Color::Green | c @ Color::Blue | c @ Color::Rgb(_, _, _) | c @ Color::Cyan => "Not red",
};
match color {
Color::Rgb(r, _, _) if r > 0 => "Some red",
Color::Red | Color::Green | Color::Blue | Color::Rgb(..) | Color::Cyan => "No red",
};
match color {
Color::Red | Color::Green | Color::Blue | Color::Cyan => {},
Color::Rgb(..) => {},
};
let x: u8 = unimplemented!();
match x {
0 => {},
140 => {},
_ => {},
};
}
3 changes: 3 additions & 0 deletions tests/ui/wildcard_enum_match_arm.rs
@@ -1,4 +1,7 @@
// run-rustfix

#![deny(clippy::wildcard_enum_match_arm)]
#![allow(unreachable_code, unused_variables)]

#[derive(Clone, Copy, Debug, Eq, PartialEq)]
enum Color {
Expand Down
10 changes: 5 additions & 5 deletions tests/ui/wildcard_enum_match_arm.stderr
@@ -1,29 +1,29 @@
error: wildcard match will miss any future added variants.
--> $DIR/wildcard_enum_match_arm.rs:26:9
--> $DIR/wildcard_enum_match_arm.rs:29:9
|
LL | _ => eprintln!("Not red"),
| ^ help: try this: `Color::Green | Color::Blue | Color::Rgb(..) | Color::Cyan`
|
note: lint level defined here
--> $DIR/wildcard_enum_match_arm.rs:1:9
--> $DIR/wildcard_enum_match_arm.rs:3:9
|
LL | #![deny(clippy::wildcard_enum_match_arm)]
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

error: wildcard match will miss any future added variants.
--> $DIR/wildcard_enum_match_arm.rs:30:9
--> $DIR/wildcard_enum_match_arm.rs:33:9
|
LL | _not_red => eprintln!("Not red"),
| ^^^^^^^^ help: try this: `_not_red @ Color::Green | _not_red @ Color::Blue | _not_red @ Color::Rgb(..) | _not_red @ Color::Cyan`

error: wildcard match will miss any future added variants.
--> $DIR/wildcard_enum_match_arm.rs:34:9
--> $DIR/wildcard_enum_match_arm.rs:37:9
|
LL | not_red => format!("{:?}", not_red),
| ^^^^^^^ help: try this: `not_red @ Color::Green | not_red @ Color::Blue | not_red @ Color::Rgb(..) | not_red @ Color::Cyan`

error: wildcard match will miss any future added variants.
--> $DIR/wildcard_enum_match_arm.rs:50:9
--> $DIR/wildcard_enum_match_arm.rs:53:9
|
LL | _ => "No red",
| ^ help: try this: `Color::Red | Color::Green | Color::Blue | Color::Rgb(..) | Color::Cyan`
Expand Down

0 comments on commit afd7b18

Please sign in to comment.