#![feature(plugin)] #![feature(exclusive_range_pattern)] #![plugin(clippy)] #![deny(clippy)] #![allow(unused, if_let_redundant_pattern_matching)] #![deny(single_match_else)] use std::borrow::Cow; enum Foo { Bar, Baz(u8) } use Foo::*; enum ExprNode { ExprAddrOf, Butterflies, Unicorns, } static NODE: ExprNode = ExprNode::Unicorns; fn dummy() { } fn unwrap_addr() -> Option<&'static ExprNode> { match ExprNode::Butterflies { ExprNode::ExprAddrOf => Some(&NODE), _ => { let x = 5; None }, } } fn single_match(){ let x = Some(1u8); match x { Some(y) => { println!("{:?}", y); } _ => () }; let z = (1u8,1u8); match z { (2...3, 7...9) => dummy(), _ => {} }; // Not linted (pattern guards used) match x { Some(y) if y == 0 => println!("{:?}", y), _ => () } // Not linted (no block with statements in the single arm) match z { (2...3, 7...9) => println!("{:?}", z), _ => println!("nope"), } } fn single_match_know_enum() { let x = Some(1u8); let y : Result<_, i8> = Ok(1i8); match x { Some(y) => dummy(), None => () }; match y { Ok(y) => dummy(), Err(..) => () }; let c = Cow::Borrowed(""); match c { Cow::Borrowed(..) => dummy(), Cow::Owned(..) => (), }; let z = Foo::Bar; // no warning match z { Bar => println!("42"), Baz(_) => (), } match z { Baz(_) => println!("42"), Bar => (), } } fn match_bool() { let test: bool = true; match test { true => 0, false => 42, }; let option = 1; match option == 1 { true => 1, false => 0, }; match test { true => (), false => { println!("Noooo!"); } }; match test { false => { println!("Noooo!"); } _ => (), }; match test && test { false => { println!("Noooo!"); } _ => (), }; match test { false => { println!("Noooo!"); } true => { println!("Yes!"); } }; // Not linted match option { 1 ... 10 => 1, 11 ... 20 => 2, _ => 3, }; } fn ref_pats() { { let v = &Some(0); match v { &Some(v) => println!("{:?}", v), &None => println!("none"), } match v { // this doesn't trigger, we have a different pattern &Some(v) => println!("some"), other => println!("other"), } } let tup =& (1, 2); match tup { &(v, 1) => println!("{}", v), _ => println!("none"), } // special case: using & both in expr and pats let w = Some(0); match &w { &Some(v) => println!("{:?}", v), &None => println!("none"), } // false positive: only wildcard pattern let w = Some(0); match w { _ => println!("none"), } let a = &Some(0); if let &None = a { println!("none"); } let b = Some(0); if let &None = &b { println!("none"); } } fn overlapping() { const FOO : u64 = 2; match 42 { 0 ... 10 => println!("0 ... 10"), 0 ... 11 => println!("0 ... 11"), _ => (), } match 42 { 0 ... 5 => println!("0 ... 5"), 6 ... 7 => println!("6 ... 7"), FOO ... 11 => println!("0 ... 11"), _ => (), } match 42 { 2 => println!("2"), 0 ... 5 => println!("0 ... 5"), _ => (), } match 42 { 2 => println!("2"), 0 ... 2 => println!("0 ... 2"), _ => (), } match 42 { 0 ... 10 => println!("0 ... 10"), 11 ... 50 => println!("11 ... 50"), _ => (), } match 42 { 2 => println!("2"), 0 .. 2 => println!("0 .. 2"), _ => (), } match 42 { 0 .. 10 => println!("0 .. 10"), 10 .. 50 => println!("10 .. 50"), _ => (), } match 42 { 0 .. 11 => println!("0 .. 11"), 0 ... 11 => println!("0 ... 11"), _ => (), } if let None = Some(42) { // nothing } else if let None = Some(42) { // another nothing :-) } } fn match_wild_err_arm() { let x: Result = Ok(3); match x { Ok(3) => println!("ok"), Ok(_) => println!("ok"), Err(_) => panic!("err") } match x { Ok(3) => println!("ok"), Ok(_) => println!("ok"), Err(_) => {panic!()} } match x { Ok(3) => println!("ok"), Ok(_) => println!("ok"), Err(_) => {panic!();} } // allowed when not with `panic!` block match x { Ok(3) => println!("ok"), Ok(_) => println!("ok"), Err(_) => println!("err") } // allowed when used with `unreachable!` match x { Ok(3) => println!("ok"), Ok(_) => println!("ok"), Err(_) => {unreachable!()} } match x { Ok(3) => println!("ok"), Ok(_) => println!("ok"), Err(_) => unreachable!() } match x { Ok(3) => println!("ok"), Ok(_) => println!("ok"), Err(_) => {unreachable!();} } } fn main() { }