6dcec6ae86
This splits up clippy::collapsible_if into collapsible_if for
if x {
if y { }
}
=>
if x && y { }
and collapsible_else_if for
if x {
} else {
if y { }
}
=>
if x {
} else if y {
}
so that we can lint for only the latter but not the first if we desire.
changelog: collapsible_if: split up linting for if x {} else { if y {} } into collapsible_else_if lint
69 lines
1.3 KiB
Rust
69 lines
1.3 KiB
Rust
// run-rustfix
|
|
#![allow(clippy::assertions_on_constants)]
|
|
|
|
#[rustfmt::skip]
|
|
#[warn(clippy::collapsible_if)]
|
|
#[warn(clippy::collapsible_else_if)]
|
|
|
|
fn main() {
|
|
let x = "hello";
|
|
let y = "world";
|
|
// Collapse `else { if .. }` to `else if ..`
|
|
if x == "hello" {
|
|
print!("Hello ");
|
|
} else if y == "world" {
|
|
println!("world!")
|
|
}
|
|
|
|
if x == "hello" {
|
|
print!("Hello ");
|
|
} else if let Some(42) = Some(42) {
|
|
println!("world!")
|
|
}
|
|
|
|
if x == "hello" {
|
|
print!("Hello ");
|
|
} else if y == "world" {
|
|
println!("world")
|
|
}
|
|
else {
|
|
println!("!")
|
|
}
|
|
|
|
if x == "hello" {
|
|
print!("Hello ");
|
|
} else if let Some(42) = Some(42) {
|
|
println!("world")
|
|
}
|
|
else {
|
|
println!("!")
|
|
}
|
|
|
|
if let Some(42) = Some(42) {
|
|
print!("Hello ");
|
|
} else if let Some(42) = Some(42) {
|
|
println!("world")
|
|
}
|
|
else {
|
|
println!("!")
|
|
}
|
|
|
|
if let Some(42) = Some(42) {
|
|
print!("Hello ");
|
|
} else if x == "hello" {
|
|
println!("world")
|
|
}
|
|
else {
|
|
println!("!")
|
|
}
|
|
|
|
if let Some(42) = Some(42) {
|
|
print!("Hello ");
|
|
} else if let Some(42) = Some(42) {
|
|
println!("world")
|
|
}
|
|
else {
|
|
println!("!")
|
|
}
|
|
}
|