| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:7:5 |
| | |
| LL | / if a.is_none() { |
| LL | | |
| LL | | return None; |
| LL | | } |
| | |_____^ help: replace it with: `a?;` |
| | |
| = note: `-D clippy::question-mark` implied by `-D warnings` |
| = help: to override `-D warnings` add `#[allow(clippy::question_mark)]` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:53:9 |
| | |
| LL | / if (self.opt).is_none() { |
| LL | | |
| LL | | return None; |
| LL | | } |
| | |_________^ help: replace it with: `(self.opt)?;` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:58:9 |
| | |
| LL | / if self.opt.is_none() { |
| LL | | |
| LL | | return None |
| LL | | } |
| | |_________^ help: replace it with: `self.opt?;` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:63:17 |
| | |
| LL | let _ = if self.opt.is_none() { |
| | _________________^ |
| LL | | |
| LL | | return None; |
| LL | | } else { |
| LL | | self.opt |
| LL | | }; |
| | |_________^ help: replace it with: `Some(self.opt?)` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:70:17 |
| | |
| LL | let _ = if let Some(x) = self.opt { |
| | _________________^ |
| LL | | |
| LL | | x |
| LL | | } else { |
| LL | | return None; |
| LL | | }; |
| | |_________^ help: replace it with: `self.opt?` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:88:9 |
| | |
| LL | / if self.opt.is_none() { |
| LL | | |
| LL | | return None; |
| LL | | } |
| | |_________^ help: replace it with: `self.opt.as_ref()?;` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:97:9 |
| | |
| LL | / if self.opt.is_none() { |
| LL | | |
| LL | | return None; |
| LL | | } |
| | |_________^ help: replace it with: `self.opt.as_ref()?;` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:106:9 |
| | |
| LL | / if self.opt.is_none() { |
| LL | | |
| LL | | return None; |
| LL | | } |
| | |_________^ help: replace it with: `self.opt.as_ref()?;` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:114:26 |
| | |
| LL | let v: &Vec<_> = if let Some(ref v) = self.opt { |
| | __________________________^ |
| LL | | |
| LL | | v |
| LL | | } else { |
| LL | | return None; |
| LL | | }; |
| | |_________^ help: replace it with: `self.opt.as_ref()?` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:125:17 |
| | |
| LL | let v = if let Some(v) = self.opt { |
| | _________________^ |
| LL | | |
| LL | | v |
| LL | | } else { |
| LL | | return None; |
| LL | | }; |
| | |_________^ help: replace it with: `self.opt?` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:147:5 |
| | |
| LL | / if f().is_none() { |
| LL | | |
| LL | | return None; |
| LL | | } |
| | |_____^ help: replace it with: `f()?;` |
| |
| error: this `match` expression can be replaced with `?` |
| --> tests/ui/question_mark.rs:152:16 |
| | |
| LL | let _val = match f() { |
| | ________________^ |
| LL | | |
| LL | | Some(val) => val, |
| LL | | None => return None, |
| LL | | }; |
| | |_____^ help: try instead: `f()?` |
| |
| error: this `match` expression can be replaced with `?` |
| --> tests/ui/question_mark.rs:163:5 |
| | |
| LL | / match f() { |
| LL | | |
| LL | | Some(val) => val, |
| LL | | None => return None, |
| LL | | }; |
| | |_____^ help: try instead: `f()?` |
| |
| error: this `match` expression can be replaced with `?` |
| --> tests/ui/question_mark.rs:169:5 |
| | |
| LL | / match opt_none!() { |
| LL | | |
| LL | | Some(x) => x, |
| LL | | None => return None, |
| LL | | }; |
| | |_____^ help: try instead: `opt_none!()?` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:196:13 |
| | |
| LL | let _ = if let Ok(x) = x { x } else { return x }; |
| | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace it with: `x?` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:199:5 |
| | |
| LL | / if x.is_err() { |
| LL | | |
| LL | | return x; |
| LL | | } |
| | |_____^ help: replace it with: `x?;` |
| |
| error: this `match` expression can be replaced with `?` |
| --> tests/ui/question_mark.rs:204:16 |
| | |
| LL | let _val = match func_returning_result() { |
| | ________________^ |
| LL | | |
| LL | | Ok(val) => val, |
| LL | | Err(err) => return Err(err), |
| LL | | }; |
| | |_____^ help: try instead: `func_returning_result()?` |
| |
| error: this `match` expression can be replaced with `?` |
| --> tests/ui/question_mark.rs:210:5 |
| | |
| LL | / match func_returning_result() { |
| LL | | |
| LL | | Ok(val) => val, |
| LL | | Err(err) => return Err(err), |
| LL | | }; |
| | |_____^ help: try instead: `func_returning_result()?` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:302:5 |
| | |
| LL | / if let Err(err) = func_returning_result() { |
| LL | | |
| LL | | return Err(err); |
| LL | | } |
| | |_____^ help: replace it with: `func_returning_result()?;` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:310:5 |
| | |
| LL | / if let Err(err) = func_returning_result() { |
| LL | | |
| LL | | return Err(err); |
| LL | | } |
| | |_____^ help: replace it with: `func_returning_result()?;` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:393:13 |
| | |
| LL | / if a.is_none() { |
| LL | | |
| LL | | return None; |
| ... | |
| LL | | } |
| | |_____________^ help: replace it with: `a?;` |
| |
| error: this `let...else` may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:454:5 |
| | |
| LL | / let Some(v) = bar.foo.owned.clone() else { |
| LL | | return None; |
| LL | | }; |
| | |______^ help: replace it with: `let v = bar.foo.owned.clone()?;` |
| |
| error: this `let...else` may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:469:5 |
| | |
| LL | / let Some(ref x) = foo.opt_x else { |
| LL | | return None; |
| LL | | }; |
| | |______^ help: replace it with: `let x = foo.opt_x.as_ref()?;` |
| |
| error: this `let...else` may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:479:5 |
| | |
| LL | / let Some(ref mut x) = foo.opt_x else { |
| LL | | return None; |
| LL | | }; |
| | |______^ help: replace it with: `let x = foo.opt_x.as_mut()?;` |
| |
| error: this `let...else` may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:490:5 |
| | |
| LL | / let Some(ref x @ ref y) = foo.opt_x else { |
| LL | | return None; |
| LL | | }; |
| | |______^ help: replace it with: `let x @ y = foo.opt_x.as_ref()?;` |
| |
| error: this `let...else` may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:494:5 |
| | |
| LL | / let Some(ref x @ WrapperStructWithString(_)) = bar else { |
| LL | | return None; |
| LL | | }; |
| | |______^ help: replace it with: `let x @ &WrapperStructWithString(_) = bar.as_ref()?;` |
| |
| error: this `let...else` may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:498:5 |
| | |
| LL | / let Some(ref mut x @ WrapperStructWithString(_)) = bar else { |
| LL | | return None; |
| LL | | }; |
| | |______^ help: replace it with: `let x @ &mut WrapperStructWithString(_) = bar.as_mut()?;` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:520:5 |
| | |
| LL | / if arg.is_none() { |
| LL | | |
| LL | | return None; |
| LL | | } |
| | |_____^ help: replace it with: `arg?;` |
| |
| error: this `match` expression can be replaced with `?` |
| --> tests/ui/question_mark.rs:524:15 |
| | |
| LL | let val = match arg { |
| | _______________^ |
| LL | | |
| LL | | Some(val) => val, |
| LL | | None => return None, |
| LL | | }; |
| | |_____^ help: try instead: `arg?` |
| |
| error: this `let...else` may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:534:5 |
| | |
| LL | / let Some(a) = *a else { |
| LL | | return None; |
| LL | | }; |
| | |______^ help: replace it with: `let a = (*a)?;` |
| |
| error: this `match` expression can be replaced with `?` |
| --> tests/ui/question_mark.rs:566:5 |
| | |
| LL | / match some_result { |
| LL | | |
| LL | | Ok(val) => val, |
| LL | | Err(err) => return Err(err.into()), |
| LL | | }; |
| | |_____^ help: try instead: `some_result?` |
| |
| error: this `match` expression can be replaced with `?` |
| --> tests/ui/question_mark.rs:572:5 |
| | |
| LL | / match some_result { |
| LL | | |
| LL | | Ok(val) => val, |
| LL | | Err(err) => return Err(Into::into(err)), |
| LL | | }; |
| | |_____^ help: try instead: `some_result?` |
| |
| error: this `match` expression can be replaced with `?` |
| --> tests/ui/question_mark.rs:578:5 |
| | |
| LL | / match some_result { |
| LL | | |
| LL | | Ok(val) => val, |
| LL | | Err(err) => return Err(<&str as Into<String>>::into(err)), |
| LL | | }; |
| | |_____^ help: try instead: `some_result?` |
| |
| error: this `match` expression can be replaced with `?` |
| --> tests/ui/question_mark.rs:596:17 |
| | |
| LL | let x = match result { |
| | _________________^ |
| LL | | |
| LL | | Ok(v) => v, |
| LL | | Err(e) => return Err(e), |
| LL | | }; |
| | |_________^ help: try instead: `result?` |
| |
| error: this block may be rewritten with the `?` operator |
| --> tests/ui/question_mark.rs:609:9 |
| | |
| LL | / if let Err(reason) = result { |
| LL | | |
| LL | | return Err(reason); |
| LL | | } |
| | |_________^ help: replace it with: `result?;` |
| |
| error: aborting due to 35 previous errors |
| |