2015-08-29 04:41:06 -05:00
|
|
|
#![feature(plugin)]
|
|
|
|
#![plugin(clippy)]
|
|
|
|
|
2015-10-16 13:27:13 -05:00
|
|
|
#![deny(while_let_loop, empty_loop, while_let_on_iterator)]
|
2015-11-18 05:35:18 -06:00
|
|
|
#![allow(dead_code, unused, cyclomatic_complexity)]
|
2015-10-02 02:55:34 -05:00
|
|
|
|
2015-08-29 04:41:06 -05:00
|
|
|
fn main() {
|
|
|
|
let y = Some(true);
|
|
|
|
loop { //~ERROR
|
|
|
|
if let Some(_x) = y {
|
|
|
|
let _v = 1;
|
|
|
|
} else {
|
|
|
|
break
|
|
|
|
}
|
|
|
|
}
|
|
|
|
loop { // no error, break is not in else clause
|
|
|
|
if let Some(_x) = y {
|
|
|
|
let _v = 1;
|
|
|
|
}
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
loop { //~ERROR
|
|
|
|
match y {
|
|
|
|
Some(_x) => true,
|
|
|
|
None => break
|
|
|
|
};
|
|
|
|
}
|
2015-09-27 02:39:42 -05:00
|
|
|
loop { //~ERROR
|
|
|
|
let x = match y {
|
|
|
|
Some(x) => x,
|
2015-08-29 04:41:06 -05:00
|
|
|
None => break
|
|
|
|
};
|
2015-09-27 02:39:42 -05:00
|
|
|
let _x = x;
|
|
|
|
let _str = "foo";
|
2015-08-29 04:41:06 -05:00
|
|
|
}
|
2015-10-11 11:49:01 -05:00
|
|
|
loop { //~ERROR
|
|
|
|
let x = match y {
|
|
|
|
Some(x) => x,
|
|
|
|
None => break,
|
|
|
|
};
|
|
|
|
{ let _a = "bar"; };
|
|
|
|
{ let _b = "foobar"; }
|
|
|
|
}
|
2015-08-29 04:41:06 -05:00
|
|
|
loop { // no error, else branch does something other than break
|
|
|
|
match y {
|
|
|
|
Some(_x) => true,
|
|
|
|
_ => {
|
|
|
|
let _z = 1;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
};
|
|
|
|
}
|
|
|
|
while let Some(x) = y { // no error, obviously
|
|
|
|
println!("{}", x);
|
|
|
|
}
|
2015-10-16 13:27:13 -05:00
|
|
|
|
2015-10-22 16:16:58 -05:00
|
|
|
let mut iter = 1..20;
|
|
|
|
while let Option::Some(x) = iter.next() { //~ERROR this loop could be written as a `for` loop
|
2015-10-16 13:27:13 -05:00
|
|
|
println!("{}", x);
|
|
|
|
}
|
|
|
|
|
2015-10-22 16:16:58 -05:00
|
|
|
let mut iter = 1..20;
|
|
|
|
while let Some(x) = iter.next() { //~ERROR this loop could be written as a `for` loop
|
2015-10-16 13:27:13 -05:00
|
|
|
println!("{}", x);
|
|
|
|
}
|
|
|
|
|
2015-10-22 16:16:58 -05:00
|
|
|
let mut iter = 1..20;
|
|
|
|
while let Some(_) = iter.next() {} //~ERROR this loop could be written as a `for` loop
|
2015-10-16 13:27:13 -05:00
|
|
|
|
2015-10-22 16:16:58 -05:00
|
|
|
let mut iter = 1..20;
|
|
|
|
while let None = iter.next() {} // this is fine (if nonsensical)
|
2015-10-16 13:27:13 -05:00
|
|
|
|
2015-10-22 16:16:58 -05:00
|
|
|
let mut iter = 1..20;
|
|
|
|
if let Some(x) = iter.next() { // also fine
|
2015-10-16 13:27:13 -05:00
|
|
|
println!("{}", x)
|
|
|
|
}
|
2015-10-19 18:04:21 -05:00
|
|
|
|
|
|
|
// the following shouldn't warn because it can't be written with a for loop
|
|
|
|
let mut iter = 1u32..20;
|
|
|
|
while let Some(x) = iter.next() {
|
|
|
|
println!("next: {:?}", iter.next())
|
|
|
|
}
|
2015-10-26 17:49:37 -05:00
|
|
|
|
|
|
|
// neither can this
|
|
|
|
let mut iter = 1u32..20;
|
|
|
|
while let Some(x) = iter.next() {
|
|
|
|
println!("next: {:?}", iter.next());
|
|
|
|
}
|
|
|
|
|
|
|
|
// or this
|
|
|
|
let mut iter = 1u32..20;
|
|
|
|
while let Some(x) = iter.next() {break;}
|
|
|
|
println!("Remaining iter {:?}", iter);
|
|
|
|
|
|
|
|
// or this
|
|
|
|
let mut iter = 1u32..20;
|
|
|
|
while let Some(x) = iter.next() {
|
|
|
|
iter = 1..20;
|
|
|
|
}
|
2015-08-29 04:41:06 -05:00
|
|
|
}
|
2015-10-02 02:55:34 -05:00
|
|
|
|
|
|
|
// regression test (#360)
|
|
|
|
// this should not panic
|
|
|
|
// it's okay if further iterations of the lint
|
|
|
|
// cause this function to trigger it
|
|
|
|
fn no_panic<T>(slice: &[T]) {
|
|
|
|
let mut iter = slice.iter();
|
2015-10-11 11:49:01 -05:00
|
|
|
loop { //~ERROR
|
2015-10-02 02:55:34 -05:00
|
|
|
let _ = match iter.next() {
|
|
|
|
Some(ele) => ele,
|
|
|
|
None => break
|
|
|
|
};
|
2015-10-12 06:38:18 -05:00
|
|
|
loop {} //~ERROR empty `loop {}` detected.
|
2015-10-02 02:55:34 -05:00
|
|
|
}
|
|
|
|
}
|