Found wildcard in an or-pattern RS-W1037
Anti-pattern
Minor
6 months ago
—
2 years old
The other patterns in this pattern are matched by the wildcard
55 3 => 50,
56 4 => 100,
57 5 => 250,
58 6 | _ => usize::MAX,59 };
60
61 if ssrc_state.active_user_set.remove(&ssrc).is_some()
Description
Wildcard patterns will cover all other patterns. Using other patterns in combination with the wildcard pattern is redundant.
Consider reducing this or-pattern to just a wildcard, to reduce complexity.
Bad practice
match s {
"foo" => { /* ... */ },
"bar" => { /* ... */ },
"baz" | _ => { /* ... */ }
}
Recommended
match s {
"foo" => { /* ... */ },
"bar" => { /* ... */ },
_ => { /* ... */ }
}