// run-rustfix #![allow(unreachable_code)] #![allow(clippy::unnecessary_wraps)] fn some_func(a: Option) -> Option { if a.is_none() { return None; } a } fn some_other_func(a: Option) -> Option { if a.is_none() { return None; } else { return Some(0); } unreachable!() } pub enum SeemsOption { Some(T), None, } impl SeemsOption { pub fn is_none(&self) -> bool { match *self { SeemsOption::None => true, SeemsOption::Some(_) => false, } } } fn returns_something_similar_to_option(a: SeemsOption) -> SeemsOption { if a.is_none() { return SeemsOption::None; } a } pub struct CopyStruct { pub opt: Option, } impl CopyStruct { #[rustfmt::skip] pub fn func(&self) -> Option { if (self.opt).is_none() { return None; } if self.opt.is_none() { return None } let _ = if self.opt.is_none() { return None; } else { self.opt }; let _ = if let Some(x) = self.opt { x } else { return None; }; self.opt } } #[derive(Clone)] pub struct MoveStruct { pub opt: Option>, } impl MoveStruct { pub fn ref_func(&self) -> Option> { if self.opt.is_none() { return None; } self.opt.clone() } pub fn mov_func_reuse(self) -> Option> { if self.opt.is_none() { return None; } self.opt } pub fn mov_func_no_use(self) -> Option> { if self.opt.is_none() { return None; } Some(Vec::new()) } pub fn if_let_ref_func(self) -> Option> { let v: &Vec<_> = if let Some(ref v) = self.opt { v } else { return None; }; Some(v.clone()) } pub fn if_let_mov_func(self) -> Option> { let v = if let Some(v) = self.opt { v } else { return None; }; Some(v) } } fn func() -> Option { fn f() -> Option { Some(String::new()) } if f().is_none() { return None; } Some(0) } fn func_returning_result() -> Result { Ok(1) } fn result_func(x: Result) -> Result { let _ = if let Ok(x) = x { x } else { return x }; if x.is_err() { return x; } // No warning let y = if let Ok(x) = x { x } else { return Err(0); }; // issue #7859 // no warning let _ = if let Ok(x) = func_returning_result() { x } else { return Err(0); }; // no warning if func_returning_result().is_err() { return func_returning_result(); } Ok(y) } fn main() { some_func(Some(42)); some_func(None); some_other_func(Some(42)); let copy_struct = CopyStruct { opt: Some(54) }; copy_struct.func(); let move_struct = MoveStruct { opt: Some(vec![42, 1337]), }; move_struct.ref_func(); move_struct.clone().mov_func_reuse(); move_struct.mov_func_no_use(); let so = SeemsOption::Some(45); returns_something_similar_to_option(so); func(); let _ = result_func(Ok(42)); }