blob: 1e2b93adffade1c4df3c65956f8a70e446a5f8a6 [file]
error: using `Option.and_then(Some)`, which is a no-op
--> tests/ui/bind_instead_of_map.rs:8:13
|
LL | let _ = x.and_then(Some);
| ^^^^^^^^^^^^^^^^ help: use the expression directly: `x`
|
note: the lint level is defined here
--> tests/ui/bind_instead_of_map.rs:1:9
|
LL | #![deny(clippy::bind_instead_of_map)]
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^
error: using `Option.and_then(|x| Some(y))`, which is more succinctly expressed as `map(|x| y)`
--> tests/ui/bind_instead_of_map.rs:10:13
|
LL | let _ = x.and_then(|o| Some(o + 1));
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `x.map(|o| o + 1)`
error: using `Result.and_then(Ok)`, which is a no-op
--> tests/ui/bind_instead_of_map.rs:17:13
|
LL | let _ = x.and_then(Ok);
| ^^^^^^^^^^^^^^ help: use the expression directly: `x`
error: using `Result.or_else(|x| Err(y))`, which is more succinctly expressed as `map_err(|x| y)`
--> tests/ui/bind_instead_of_map.rs:32:31
|
LL | let _: Result<u32, u32> = res.or_else(|_| if b { panic!("should not happen") } else { Err(42) });
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: use `map_err` instead
|
LL - let _: Result<u32, u32> = res.or_else(|_| if b { panic!("should not happen") } else { Err(42) });
LL + let _: Result<u32, u32> = res.map_err(|_| if b { panic!("should not happen") } else { 42 });
|
error: aborting due to 4 previous errors