Split out function for --classify character

src/output/file_name.rs

@@ -76,23 +76,32 @@ impl<'a, 'dir> FileName<'a, 'dir> {
                     // Do nothing -- the error gets displayed on the next line
                 }
             }
-        } else if classify {
+        }
-            if self.file.is_executable_file() {
+        else if classify {
-                bits.push(Style::default().paint("*"));
+            if let Some(class) = self.classify_char() {
-            } else if self.file.is_directory() {
+                bits.push(Style::default().paint(class));
-                bits.push(Style::default().paint("/"));
-            } else if self.file.is_pipe() {
-                bits.push(Style::default().paint("|"));
-            } else if self.file.is_link() {
-                bits.push(Style::default().paint("@"));
-            } else if self.file.is_socket() {
-                bits.push(Style::default().paint("="));
             }
         }
 
         bits.into()
     }
 
+    fn classify_char(&self) -> Option<&'static str> {
+        if self.file.is_executable_file() {
+            Some("*")
+        } else if self.file.is_directory() {
+            Some("/")
+        } else if self.file.is_pipe() {
+            Some("|")
+        } else if self.file.is_link() {
+            Some("@")
+        } else if self.file.is_socket() {
+            Some("=")
+        } else {
+            None
+        }
+    }
+
     /// Returns at least one ANSI-highlighted string representing this file’s
     /// name using the given set of colours.
     ///