blob: 0f9466a6a6b305e74fcfe46d90e3eca0267e8ed5 [file] [log] [blame]
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:13:13
|
LL | let _ = Some(5).map_or(false, |n| n == 5);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= note: `-D clippy::unnecessary-map-or` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(clippy::unnecessary_map_or)]`
help: use a standard comparison instead
|
LL - let _ = Some(5).map_or(false, |n| n == 5);
LL + let _ = Some(5) == Some(5);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:15:13
|
LL | let _ = Some(5).map_or(true, |n| n != 5);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use a standard comparison instead
|
LL - let _ = Some(5).map_or(true, |n| n != 5);
LL + let _ = Some(5) != Some(5);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:17:13
|
LL | let _ = Some(5).map_or(false, |n| {
| _____________^
LL | |
LL | | let _ = 1;
LL | | n == 5
LL | | });
| |______^
|
help: use a standard comparison instead
|
LL - let _ = Some(5).map_or(false, |n| {
LL -
LL - let _ = 1;
LL - n == 5
LL - });
LL + let _ = Some(5) == Some(5);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:22:13
|
LL | let _ = Some(5).map_or(false, |n| {
| _____________^
LL | |
LL | | let _ = n;
LL | | 6 >= 5
LL | | });
| |______^
|
help: use is_some_and instead
|
LL - let _ = Some(5).map_or(false, |n| {
LL + let _ = Some(5).is_some_and(|n| {
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:27:13
|
LL | let _ = Some(vec![5]).map_or(false, |n| n == [5]);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use is_some_and instead
|
LL - let _ = Some(vec![5]).map_or(false, |n| n == [5]);
LL + let _ = Some(vec![5]).is_some_and(|n| n == [5]);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:29:13
|
LL | let _ = Some(vec![1]).map_or(false, |n| vec![2] == n);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use is_some_and instead
|
LL - let _ = Some(vec![1]).map_or(false, |n| vec![2] == n);
LL + let _ = Some(vec![1]).is_some_and(|n| vec![2] == n);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:31:13
|
LL | let _ = Some(5).map_or(false, |n| n == n);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use is_some_and instead
|
LL - let _ = Some(5).map_or(false, |n| n == n);
LL + let _ = Some(5).is_some_and(|n| n == n);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:33:13
|
LL | let _ = Some(5).map_or(false, |n| n == if 2 > 1 { n } else { 0 });
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use is_some_and instead
|
LL - let _ = Some(5).map_or(false, |n| n == if 2 > 1 { n } else { 0 });
LL + let _ = Some(5).is_some_and(|n| n == if 2 > 1 { n } else { 0 });
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:35:13
|
LL | let _ = Ok::<Vec<i32>, i32>(vec![5]).map_or(false, |n| n == [5]);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use is_ok_and instead
|
LL - let _ = Ok::<Vec<i32>, i32>(vec![5]).map_or(false, |n| n == [5]);
LL + let _ = Ok::<Vec<i32>, i32>(vec![5]).is_ok_and(|n| n == [5]);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:37:13
|
LL | let _ = Ok::<i32, i32>(5).map_or(false, |n| n == 5);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use a standard comparison instead
|
LL - let _ = Ok::<i32, i32>(5).map_or(false, |n| n == 5);
LL + let _ = Ok::<i32, i32>(5) == Ok(5);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:39:13
|
LL | let _ = Some(5).map_or(false, |n| n == 5).then(|| 1);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use a standard comparison instead
|
LL - let _ = Some(5).map_or(false, |n| n == 5).then(|| 1);
LL + let _ = (Some(5) == Some(5)).then(|| 1);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:41:13
|
LL | let _ = Some(5).map_or(true, |n| n == 5);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use is_none_or instead
|
LL - let _ = Some(5).map_or(true, |n| n == 5);
LL + let _ = Some(5).is_none_or(|n| n == 5);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:43:13
|
LL | let _ = Some(5).map_or(true, |n| 5 == n);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use is_none_or instead
|
LL - let _ = Some(5).map_or(true, |n| 5 == n);
LL + let _ = Some(5).is_none_or(|n| 5 == n);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:45:14
|
LL | let _ = !Some(5).map_or(false, |n| n == 5);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use a standard comparison instead
|
LL - let _ = !Some(5).map_or(false, |n| n == 5);
LL + let _ = !(Some(5) == Some(5));
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:47:13
|
LL | let _ = Some(5).map_or(false, |n| n == 5) || false;
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use a standard comparison instead
|
LL - let _ = Some(5).map_or(false, |n| n == 5) || false;
LL + let _ = (Some(5) == Some(5)) || false;
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:49:13
|
LL | let _ = Some(5).map_or(false, |n| n == 5) as usize;
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use a standard comparison instead
|
LL - let _ = Some(5).map_or(false, |n| n == 5) as usize;
LL + let _ = (Some(5) == Some(5)) as usize;
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:74:13
|
LL | let _ = r.map_or(false, |x| x == 7);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use is_ok_and instead
|
LL - let _ = r.map_or(false, |x| x == 7);
LL + let _ = r.is_ok_and(|x| x == 7);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:80:13
|
LL | let _ = r.map_or(false, func);
| ^^^^^^^^^^^^^^^^^^^^^
|
help: use is_ok_and instead
|
LL - let _ = r.map_or(false, func);
LL + let _ = r.is_ok_and(func);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:82:13
|
LL | let _ = Some(5).map_or(false, func);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use is_some_and instead
|
LL - let _ = Some(5).map_or(false, func);
LL + let _ = Some(5).is_some_and(func);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:84:13
|
LL | let _ = Some(5).map_or(true, func);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use is_none_or instead
|
LL - let _ = Some(5).map_or(true, func);
LL + let _ = Some(5).is_none_or(func);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:90:13
|
LL | let _ = r.map_or(false, |x| x == 8);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use a standard comparison instead
|
LL - let _ = r.map_or(false, |x| x == 8);
LL + let _ = r == Ok(8);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:111:5
|
LL | o.map_or(true, |n| n > 5) || (o as &Option<u32>).map_or(true, |n| n < 5)
| ^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use is_none_or instead
|
LL - o.map_or(true, |n| n > 5) || (o as &Option<u32>).map_or(true, |n| n < 5)
LL + o.is_none_or(|n| n > 5) || (o as &Option<u32>).map_or(true, |n| n < 5)
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:111:34
|
LL | o.map_or(true, |n| n > 5) || (o as &Option<u32>).map_or(true, |n| n < 5)
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use is_none_or instead
|
LL - o.map_or(true, |n| n > 5) || (o as &Option<u32>).map_or(true, |n| n < 5)
LL + o.map_or(true, |n| n > 5) || (o as &Option<u32>).is_none_or(|n| n < 5)
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:126:5
|
LL | o.map_or(true, |n| n > 5)
| ^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use is_none_or instead
|
LL - o.map_or(true, |n| n > 5)
LL + o.is_none_or(|n| n > 5)
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:131:13
|
LL | let x = a.map_or(false, |a| a == *s);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use is_some_and instead
|
LL - let x = a.map_or(false, |a| a == *s);
LL + let x = a.is_some_and(|a| a == *s);
|
error: this `map_or` can be simplified
--> tests/ui/unnecessary_map_or.rs:133:13
|
LL | let y = b.map_or(true, |b| b == *s);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use is_none_or instead
|
LL - let y = b.map_or(true, |b| b == *s);
LL + let y = b.is_none_or(|b| b == *s);
|
error: aborting due to 26 previous errors