src/tools/clippy/tests/ui/bind_instead_of_map_multipart.stderr

   1 error: using `Option.and_then(|x| Some(y))`, which is more succinctly expressed as `map(|x| y)`
   2   --> $DIR/bind_instead_of_map_multipart.rs:5:13
   3    |
   4 LL |     let _ = Some("42").and_then(|s| if s.len() < 42 { Some(0) } else { Some(s.len()) });
   5    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   6    |
   7 note: the lint level is defined here
   8   --> $DIR/bind_instead_of_map_multipart.rs:1:9
   9    |
  10 LL | #![deny(clippy::bind_instead_of_map)]
  11    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  12 help: try this
  13    |
  14 LL |     let _ = Some("42").map(|s| if s.len() < 42 { 0 } else { s.len() });
  15    |                        ~~~                       ~          ~~~~~~~
  16
  17 error: using `Result.and_then(|x| Ok(y))`, which is more succinctly expressed as `map(|x| y)`
  18   --> $DIR/bind_instead_of_map_multipart.rs:8:13
  19    |
  20 LL |     let _ = Ok::<_, ()>("42").and_then(|s| if s.len() < 42 { Ok(0) } else { Ok(s.len()) });
  21    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  22    |
  23 help: try this
  24    |
  25 LL |     let _ = Ok::<_, ()>("42").map(|s| if s.len() < 42 { 0 } else { s.len() });
  26    |                               ~~~                       ~          ~~~~~~~
  27
  28 error: using `Result.or_else(|x| Err(y))`, which is more succinctly expressed as `map_err(|x| y)`
  29   --> $DIR/bind_instead_of_map_multipart.rs:11:13
  30    |
  31 LL |     let _ = Err::<(), _>("42").or_else(|s| if s.len() < 42 { Err(s.len() + 20) } else { Err(s.len()) });
  32    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  33    |
  34 help: try this
  35    |
  36 LL |     let _ = Err::<(), _>("42").map_err(|s| if s.len() < 42 { s.len() + 20 } else { s.len() });
  37    |                                ~~~~~~~                       ~~~~~~~~~~~~          ~~~~~~~
  38
  39 error: using `Option.and_then(|x| Some(y))`, which is more succinctly expressed as `map(|x| y)`
  40   --> $DIR/bind_instead_of_map_multipart.rs:19:5
  41    |
  42 LL | /     Some("42").and_then(|s| {
  43 LL | |         if {
  44 LL | |             if s == "43" {
  45 LL | |                 return Some(43);
  46 ...  |
  47 LL | |         }
  48 LL | |     });
  49    | |______^
  50    |
  51 help: try this
  52    |
  53 LL ~     Some("42").map(|s| {
  54 LL |         if {
  55 LL |             if s == "43" {
  56 LL ~                 return 43;
  57 LL |             }
  58 LL |             s == "42"
  59  ...
  60
  61 error: using `Option.and_then(|x| Some(y))`, which is more succinctly expressed as `map(|x| y)`
  62   --> $DIR/bind_instead_of_map_multipart.rs:60:13
  63    |
  64 LL |     let _ = Some("").and_then(|s| if s.len() == 20 { Some(m!()) } else { Some(Some(20)) });
  65    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  66    |
  67 help: try this
  68    |
  69 LL |     let _ = Some("").map(|s| if s.len() == 20 { m!() } else { Some(20) });
  70    |                      ~~~                        ~~~~          ~~~~~~~~
  71
  72 error: aborting due to 5 previous errors
  73