clippy lint: avoid manual implementation of Option

src/flags/color.rs

@@ -84,11 +84,8 @@ impl Configurable<Self> for ColorOption {
             return Some(Self::Never);
         }
 
-        if let Some(color) = &config.color {
+        config.color.as_ref()
-            Some(color.when)
+            .map(|color| color.when)
-        } else {
-            None
-        }
     }
 
     fn from_environment() -> Option<Self> {

src/flags/dereference.rs

@@ -29,11 +29,8 @@ impl Configurable<Self> for Dereference {
     /// If the `Config::dereference` has value, this returns its value
     /// as the value of the `Dereference`, in a [Some], Otherwise this returns [None].
     fn from_config(config: &Config) -> Option<Self> {
-        if let Some(deref) = &config.dereference {
+        config.dereference.as_ref()
-            Some(Self(*deref))
+            .map(|deref| Self(*deref))
-        } else {
-            None
-        }
     }
 }
 

src/flags/indicators.rs

@@ -30,11 +30,8 @@ impl Configurable<Self> for Indicators {
     /// this returns its value as the value of the `Indicators`, in a [Some].
     /// Otherwise this returns [None].
     fn from_config(config: &Config) -> Option<Self> {
-        if let Some(ind) = &config.indicators {
+        config.indicators.as_ref()
-            Some(Self(*ind))
+            .map(|ind| Self(*ind))
-        } else {
-            None
-        }
     }
 }
 

src/flags/symlink_arrow.rs

@@ -21,11 +21,8 @@ impl Configurable<Self> for SymlinkArrow {
     /// returns its value as the value of the `SymlinkArrow`, in a [Some].
     /// Otherwise this returns [None].
     fn from_config(config: &Config) -> Option<Self> {
-        if let Some(arrow) = &config.symlink_arrow {
+        config.symlink_arrow.as_ref()
-            Some(SymlinkArrow(arrow.to_string()))
+            .map(|arrow| SymlinkArrow(arrow.to_string()))
-        } else {
-            None
-        }
     }
 }
 

src/flags/symlinks.rs

@@ -30,11 +30,7 @@ impl Configurable<Self> for NoSymlink {
     /// this returns it as the value of the `NoSymlink`, in a [Some].
     /// Otherwise this returns [None].
     fn from_config(config: &Config) -> Option<Self> {
-        if let Some(no_link) = config.no_symlink {
+        config.no_symlink.map(Self)
-            Some(Self(no_link))
-        } else {
-            None
-        }
     }
 }
 

src/flags/total_size.rs

@@ -30,11 +30,7 @@ impl Configurable<Self> for TotalSize {
     /// this returns it as the value of the `TotalSize`, in a [Some].
     /// Otherwise this returns [None].
     fn from_config(config: &Config) -> Option<Self> {
-        if let Some(total) = config.total_size {
+        config.total_size.map(Self)
-            Some(Self(total))
-        } else {
-            None
-        }
     }
 }
 

src/meta/symlink.rs

@@ -45,11 +45,8 @@ impl<'a> From<&'a Path> for SymLink {
 
 impl SymLink {
     pub fn symlink_string(&self) -> Option<String> {
-        if let Some(ref target) = self.target {
+        self.target.as_ref()
-            Some(target.to_string())
+            .map(|target| target.to_string())
-        } else {
-            None
-        }
     }
 
     pub fn render(&self, colors: &Colors, flag: &Flags) -> ColoredString {