Cleanup

Userland/ls.cpp

@@ -11,7 +11,7 @@
 #include <AK/AKString.h>
 #include <AK/Vector.h>
 
-static int do_dir(const char* path);
+static int do_dir_long(const char* path);
 static int do_dir_short(const char* path);
 
 static bool flag_colorize = true;
@@ -45,30 +45,24 @@ int main(int argc, char** argv)
 
     int status;
     if (optind >= argc) {
-      if (flag_long) {
-          status = do_dir(".");
-      } else {
-          status = do_dir_short(".");
-      }
-      return status;
+      status = do_dir(".")
+    } if (optind+1 >= argc) {
+      status = do_dir(argv[optind]);
     } else {
-        bool show_names = !(optind+1 >= argc);
-
         for (; optind < argc; optind++) {
-          if (show_names) {
             printf("%s:\n", argv[optind]);
-          }
-          if (flag_long) {
-              status = do_dir(argv[optind]);
-          } else {
-              status = do_dir_short(argv[optind]);
-          }
-          if (status != 0) {
-            return status;
-          }
+            status = do_dir(argv[optind]);
         }
     }
-    return 0;
+    return status;
+}
+
+int do_dir(const char* path) {
+    if (flag_long) {
+        return do_dir_long(argv[optind]);
+    } else {
+        return do_dir_short(argv[optind]);
+    }
 }
 
 void get_geometry(int& rows, int& columns)
@@ -186,7 +180,7 @@ bool print_filesystem_object(const char* path, const char* name) {
     return true;
 }
 
-int do_dir(const char* path)
+int do_dir_long(const char* path)
 {
     DIR* dirp = opendir(path);
     if (!dirp) {