mirror of
https://github.com/rust-lang/rust-clippy
synced 2024-12-23 19:43:20 +00:00
1cb94a7e6c
Split out while_let_loop tests into separate file. Split ice-360 out into separate file.
119 lines
2.1 KiB
Rust
119 lines
2.1 KiB
Rust
#![warn(clippy::while_let_loop)]
|
|
|
|
fn main() {
|
|
let y = Some(true);
|
|
loop {
|
|
if let Some(_x) = y {
|
|
let _v = 1;
|
|
} else {
|
|
break;
|
|
}
|
|
}
|
|
|
|
#[allow(clippy::never_loop)]
|
|
loop {
|
|
// no error, break is not in else clause
|
|
if let Some(_x) = y {
|
|
let _v = 1;
|
|
}
|
|
break;
|
|
}
|
|
|
|
loop {
|
|
match y {
|
|
Some(_x) => true,
|
|
None => break,
|
|
};
|
|
}
|
|
|
|
loop {
|
|
let x = match y {
|
|
Some(x) => x,
|
|
None => break,
|
|
};
|
|
let _x = x;
|
|
let _str = "foo";
|
|
}
|
|
|
|
loop {
|
|
let x = match y {
|
|
Some(x) => x,
|
|
None => break,
|
|
};
|
|
{
|
|
let _a = "bar";
|
|
};
|
|
{
|
|
let _b = "foobar";
|
|
}
|
|
}
|
|
|
|
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);
|
|
}
|
|
|
|
// #675, this used to have a wrong suggestion
|
|
loop {
|
|
let (e, l) = match "".split_whitespace().next() {
|
|
Some(word) => (word.is_empty(), word.len()),
|
|
None => break,
|
|
};
|
|
|
|
let _ = (e, l);
|
|
}
|
|
}
|
|
|
|
fn issue771() {
|
|
let mut a = 100;
|
|
let b = Some(true);
|
|
loop {
|
|
if a > 10 {
|
|
break;
|
|
}
|
|
|
|
match b {
|
|
Some(_) => a = 0,
|
|
None => break,
|
|
}
|
|
}
|
|
}
|
|
|
|
fn issue1017() {
|
|
let r: Result<u32, u32> = Ok(42);
|
|
let mut len = 1337;
|
|
|
|
loop {
|
|
match r {
|
|
Err(_) => len = 0,
|
|
Ok(length) => {
|
|
len = length;
|
|
break;
|
|
},
|
|
}
|
|
}
|
|
}
|
|
|
|
#[allow(clippy::never_loop)]
|
|
fn issue1948() {
|
|
// should not trigger clippy::while_let_loop lint because break passes an expression
|
|
let a = Some(10);
|
|
let b = loop {
|
|
if let Some(c) = a {
|
|
break Some(c);
|
|
} else {
|
|
break None;
|
|
}
|
|
};
|
|
}
|