| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:10:5 |
| | |
| LL | / if let Some(x) = string { |
| LL | | (true, x) |
| LL | | } else { |
| LL | | (false, "hello") |
| LL | | } |
| | |_____^ help: try: `string.map_or((false, "hello"), |x| (true, x))` |
| | |
| = note: `-D clippy::option-if-let-else` implied by `-D warnings` |
| = help: to override `-D warnings` add `#[allow(clippy::option_if_let_else)]` |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:28:13 |
| | |
| LL | let _ = if let Some(s) = *string { s.len() } else { 0 }; |
| | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `string.map_or(0, |s| s.len())` |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:29:13 |
| | |
| LL | let _ = if let Some(s) = &num { s } else { &0 }; |
| | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `num.as_ref().map_or(&0, |s| s)` |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:30:13 |
| | |
| LL | let _ = if let Some(s) = &mut num { |
| | _____________^ |
| LL | | *s += 1; |
| LL | | s |
| LL | | } else { |
| LL | | &0 |
| LL | | }; |
| | |_____^ |
| | |
| help: try |
| | |
| LL ~ let _ = num.as_mut().map_or(&0, |s| { |
| LL + *s += 1; |
| LL + s |
| LL ~ }); |
| | |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:36:13 |
| | |
| LL | let _ = if let Some(ref s) = num { s } else { &0 }; |
| | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `num.as_ref().map_or(&0, |s| s)` |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:37:13 |
| | |
| LL | let _ = if let Some(mut s) = num { |
| | _____________^ |
| LL | | s += 1; |
| LL | | s |
| LL | | } else { |
| LL | | 0 |
| LL | | }; |
| | |_____^ |
| | |
| help: try |
| | |
| LL ~ let _ = num.map_or(0, |mut s| { |
| LL + s += 1; |
| LL + s |
| LL ~ }); |
| | |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:43:13 |
| | |
| LL | let _ = if let Some(ref mut s) = num { |
| | _____________^ |
| LL | | *s += 1; |
| LL | | s |
| LL | | } else { |
| LL | | &0 |
| LL | | }; |
| | |_____^ |
| | |
| help: try |
| | |
| LL ~ let _ = num.as_mut().map_or(&0, |s| { |
| LL + *s += 1; |
| LL + s |
| LL ~ }); |
| | |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:52:5 |
| | |
| LL | / if let Some(x) = arg { |
| LL | | let y = x * x; |
| LL | | y * y |
| LL | | } else { |
| LL | | 13 |
| LL | | } |
| | |_____^ |
| | |
| help: try |
| | |
| LL ~ arg.map_or(13, |x| { |
| LL + let y = x * x; |
| LL + y * y |
| LL + }) |
| | |
| |
| error: use Option::map_or_else instead of an if let/else |
| --> $DIR/option_if_let_else.rs:65:13 |
| | |
| LL | let _ = if let Some(x) = arg { |
| | _____________^ |
| LL | | x |
| LL | | } else { |
| LL | | // map_or_else must be suggested |
| LL | | side_effect() |
| LL | | }; |
| | |_____^ help: try: `arg.map_or_else(side_effect, |x| x)` |
| |
| error: use Option::map_or_else instead of an if let/else |
| --> $DIR/option_if_let_else.rs:74:13 |
| | |
| LL | let _ = if let Some(x) = arg { |
| | _____________^ |
| LL | | x * x * x * x |
| LL | | } else { |
| LL | | let mut y = 1; |
| ... | |
| LL | | y |
| LL | | }; |
| | |_____^ |
| | |
| help: try |
| | |
| LL ~ let _ = arg.map_or_else(|| { |
| LL + let mut y = 1; |
| LL + y = (y + 2 / y) / 2; |
| LL + y = (y + 2 / y) / 2; |
| LL + y |
| LL ~ }, |x| x * x * x * x); |
| | |
| |
| error: use Option::map_or_else instead of an if let/else |
| --> $DIR/option_if_let_else.rs:107:13 |
| | |
| LL | / if let Some(idx) = s.find('.') { |
| LL | | vec![s[..idx].to_string(), s[idx..].to_string()] |
| LL | | } else { |
| LL | | vec![s.to_string()] |
| LL | | } |
| | |_____________^ help: try: `s.find('.').map_or_else(|| vec![s.to_string()], |idx| vec![s[..idx].to_string(), s[idx..].to_string()])` |
| |
| error: use Option::map_or_else instead of an if let/else |
| --> $DIR/option_if_let_else.rs:118:5 |
| | |
| LL | / if let Ok(binding) = variable { |
| LL | | println!("Ok {binding}"); |
| LL | | true |
| LL | | } else { |
| LL | | println!("Err"); |
| LL | | false |
| LL | | } |
| | |_____^ |
| | |
| help: try |
| | |
| LL ~ variable.map_or_else(|_| { |
| LL + println!("Err"); |
| LL + false |
| LL + }, |binding| { |
| LL + println!("Ok {binding}"); |
| LL + true |
| LL + }) |
| | |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:142:13 |
| | |
| LL | let _ = if let Some(x) = optional { x + 2 } else { 5 }; |
| | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `optional.map_or(5, |x| x + 2)` |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:152:13 |
| | |
| LL | let _ = if let Some(x) = Some(0) { |
| | _____________^ |
| LL | | loop { |
| LL | | if x == 0 { |
| LL | | break x; |
| ... | |
| LL | | 0 |
| LL | | }; |
| | |_____^ |
| | |
| help: try |
| | |
| LL ~ let _ = Some(0).map_or(0, |x| loop { |
| LL + if x == 0 { |
| LL + break x; |
| LL + } |
| LL ~ }); |
| | |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:180:13 |
| | |
| LL | let _ = if let Some(x) = Some(0) { s.len() + x } else { s.len() }; |
| | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `Some(0).map_or(s.len(), |x| s.len() + x)` |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:184:13 |
| | |
| LL | let _ = if let Some(x) = Some(0) { |
| | _____________^ |
| LL | | let s = s; |
| LL | | s.len() + x |
| LL | | } else { |
| LL | | 1 |
| LL | | }; |
| | |_____^ |
| | |
| help: try |
| | |
| LL ~ let _ = Some(0).map_or(1, |x| { |
| LL + let s = s; |
| LL + s.len() + x |
| LL ~ }); |
| | |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:223:13 |
| | |
| LL | let _ = match s { |
| | _____________^ |
| LL | | Some(string) => string.len(), |
| LL | | None => 1, |
| LL | | }; |
| | |_____^ help: try: `s.map_or(1, |string| string.len())` |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:227:13 |
| | |
| LL | let _ = match Some(10) { |
| | _____________^ |
| LL | | Some(a) => a + 1, |
| LL | | None => 5, |
| LL | | }; |
| | |_____^ help: try: `Some(10).map_or(5, |a| a + 1)` |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:233:13 |
| | |
| LL | let _ = match res { |
| | _____________^ |
| LL | | Ok(a) => a + 1, |
| LL | | _ => 1, |
| LL | | }; |
| | |_____^ help: try: `res.map_or(1, |a| a + 1)` |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:237:13 |
| | |
| LL | let _ = match res { |
| | _____________^ |
| LL | | Err(_) => 1, |
| LL | | Ok(a) => a + 1, |
| LL | | }; |
| | |_____^ help: try: `res.map_or(1, |a| a + 1)` |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:241:13 |
| | |
| LL | let _ = if let Ok(a) = res { a + 1 } else { 5 }; |
| | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `res.map_or(5, |a| a + 1)` |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:258:17 |
| | |
| LL | let _ = match initial { |
| | _________________^ |
| LL | | Some(value) => do_something(value), |
| LL | | None => 42, |
| LL | | }; |
| | |_________^ help: try: `initial.as_ref().map_or(42, |value| do_something(value))` |
| |
| error: use Option::map_or instead of an if let/else |
| --> $DIR/option_if_let_else.rs:265:17 |
| | |
| LL | let _ = match initial { |
| | _________________^ |
| LL | | Some(value) => do_something2(value), |
| LL | | None => 42, |
| LL | | }; |
| | |_________^ help: try: `initial.as_mut().map_or(42, |value| do_something2(value))` |
| |
| error: use Option::map_or_else instead of an if let/else |
| --> $DIR/option_if_let_else.rs:288:24 |
| | |
| LL | let mut _hashmap = if let Some(hm) = &opt { |
| | ________________________^ |
| LL | | hm.clone() |
| LL | | } else { |
| LL | | HashMap::new() |
| LL | | }; |
| | |_____^ help: try: `opt.as_ref().map_or_else(HashMap::new, |hm| hm.clone())` |
| |
| error: use Option::map_or_else instead of an if let/else |
| --> $DIR/option_if_let_else.rs:294:19 |
| | |
| LL | let mut _hm = if let Some(hm) = &opt { hm.clone() } else { new_map!() }; |
| | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `opt.as_ref().map_or_else(|| new_map!(), |hm| hm.clone())` |
| |
| error: aborting due to 25 previous errors |
| |