rust-clippy/tests/ui/match_same_arms.rs

147 lines
3.1 KiB
Rust
Raw Normal View History

#![warn(clippy::match_same_arms)]
2018-12-09 22:26:16 +00:00
#![allow(
clippy::blacklisted_name,
clippy::collapsible_if,
clippy::cognitive_complexity,
2018-12-09 22:26:16 +00:00
clippy::eq_op,
clippy::needless_continue,
clippy::needless_return,
clippy::no_effect,
clippy::zero_divided_by_zero,
clippy::unused_unit
)]
2018-07-28 15:34:52 +00:00
2016-02-10 00:22:53 +00:00
fn bar<T>(_: T) {}
2018-12-09 22:26:16 +00:00
fn foo() -> bool {
unimplemented!()
}
pub enum Abc {
A,
B,
C,
}
2018-09-27 17:10:20 +00:00
#[allow(clippy::unused_unit)]
fn match_same_arms() {
let _ = match 42 {
42 => {
foo();
let mut a = 42 + [23].len() as i32;
if true {
a += 7;
}
2018-12-09 22:26:16 +00:00
a = -31 - a;
a
2018-12-09 22:26:16 +00:00
},
_ => {
//~ ERROR match arms have same body
foo();
let mut a = 42 + [23].len() as i32;
if true {
a += 7;
}
2018-12-09 22:26:16 +00:00
a = -31 - a;
a
2018-12-09 22:26:16 +00:00
},
};
let _ = match Abc::A {
Abc::A => 0,
Abc::B => 1,
_ => 0, //~ ERROR match arms have same body
};
2016-02-10 00:22:53 +00:00
let _ = match 42 {
42 => foo(),
51 => foo(), //~ ERROR match arms have same body
2016-02-10 00:22:53 +00:00
_ => true,
};
let _ = match Some(42) {
Some(_) => 24,
None => 24, //~ ERROR match arms have same body
};
2016-05-31 19:50:13 +00:00
let _ = match Some(42) {
Some(foo) => 24,
None => 24,
};
2016-02-10 00:22:53 +00:00
let _ = match Some(42) {
Some(42) => 24,
Some(a) => 24, // bindings are different
None => 0,
};
let _ = match Some(42) {
Some(a) if a > 0 => 24,
Some(a) => 24, // one arm has a guard
None => 0,
};
2016-02-10 00:22:53 +00:00
match (Some(42), Some(42)) {
(Some(a), None) => bar(a),
(None, Some(a)) => bar(a), //~ ERROR match arms have same body
2016-02-10 00:22:53 +00:00
_ => (),
}
match (Some(42), Some(42)) {
(Some(a), ..) => bar(a),
(.., Some(a)) => bar(a), //~ ERROR match arms have same body
_ => (),
}
match (1, 2, 3) {
(1, .., 3) => 42,
(.., 3) => 42, //~ ERROR match arms have same body
_ => 0,
};
let _ = match Some(()) {
Some(()) => 0.0,
2018-12-09 22:26:16 +00:00
None => -0.0,
};
2016-02-10 00:22:53 +00:00
match (Some(42), Some("")) {
(Some(a), None) => bar(a),
(None, Some(a)) => bar(a), // bindings have different types
_ => (),
}
let _ = match 42 {
42 => 1,
51 => 1, //~ ERROR match arms have same body
41 => 2,
52 => 2, //~ ERROR match arms have same body
_ => 0,
};
let _ = match 42 {
1 => 2,
2 => 2, //~ ERROR 2nd matched arms have same body
3 => 2, //~ ERROR 3rd matched arms have same body
4 => 3,
_ => 0,
};
}
mod issue4244 {
#[derive(PartialEq, PartialOrd, Eq, Ord)]
pub enum CommandInfo {
BuiltIn { name: String, about: Option<String> },
External { name: String, path: std::path::PathBuf },
}
impl CommandInfo {
pub fn name(&self) -> String {
match self {
CommandInfo::BuiltIn { name, .. } => name.to_string(),
CommandInfo::External { name, .. } => name.to_string(),
}
}
}
}
fn main() {}