Simplify code by using guard clauses

Author: sebromero

src/Utils.h

@@ -24,69 +24,71 @@
 
 
 [[gnu::unused]] static bool copyFolder(const char* source, const char* destination) {
-    DIR* dir = opendir(source);
-    if (dir == nullptr) {
-        printf("Failed to open source directory\n");
-        return false;
+  DIR* dir = opendir(source);
+  if (dir == nullptr) {
+    printf("Failed to open source directory\n");
+    return false;
+  }
+
+  // Create destination directory if it doesn't exist
+  if (mkdir(destination, 0777) != 0 && errno != EEXIST) {
+    printf("Failed to create destination directory\n");
+    closedir(dir);
+    return false;
+  }
+
+  struct dirent* entry;
+  while ((entry = readdir(dir)) != nullptr) {
+    // Skip "." and ".."
+    if (strcmp(entry->d_name, ".") == 0 || strcmp(entry->d_name, "..") == 0) {
+      continue;
     }
 
-    // Create destination directory if it doesn't exist
-    if (mkdir(destination, 0777) != 0 && errno != EEXIST) {
-        printf("Failed to create destination directory\n");
+    char sourcePath[PATH_MAX_LENGTH];
+    snprintf(sourcePath, PATH_MAX_LENGTH, "%s/%s", source, entry->d_name);
+
+    char destinationPath[PATH_MAX_LENGTH];
+    snprintf(destinationPath, PATH_MAX_LENGTH, "%s/%s", destination, entry->d_name);
+
+    struct stat fileInfo;
+    if (stat(sourcePath, &fileInfo) != 0) {
+      closedir(dir);
+      return false;
+    }
+
+    if (S_ISDIR(fileInfo.st_mode)) {
+      // Recursively copy subdirectories
+      if (!copyFolder(sourcePath, destinationPath)) {
         closedir(dir);
         return false;
-    }
+      }
+    } else {
+      // Copy regular files
+      FILE* sourceFile = fopen(sourcePath, "r");
+      if (sourceFile == nullptr) {
+        closedir(dir);
+        return false;
+      }
 
-    struct dirent* entry;
-    while ((entry = readdir(dir)) != nullptr) {
-        if (strcmp(entry->d_name, ".") != 0 && strcmp(entry->d_name, "..") != 0) {
-            char sourcePath[PATH_MAX_LENGTH];
-            snprintf(sourcePath, PATH_MAX_LENGTH, "%s/%s", source, entry->d_name);
-
-            char destinationPath[PATH_MAX_LENGTH];
-            snprintf(destinationPath, PATH_MAX_LENGTH, "%s/%s", destination, entry->d_name);
-
-            struct stat fileInfo;
-            if (stat(sourcePath, &fileInfo) != 0) {
-                closedir(dir);
-                return false;
-            }
-
-            if (S_ISDIR(fileInfo.st_mode)) {
-                // Recursively copy subdirectories
-                if (!copyFolder(sourcePath, destinationPath)) {
-                    closedir(dir);
-                    return false;
-                }
-            } else {
-                // Copy regular files
-                FILE* sourceFile = fopen(sourcePath, "r");
-                if (sourceFile == nullptr) {
-
-                    closedir(dir);
-                    return false;
-                }
-
-                FILE* destinationFile = fopen(destinationPath, "w");
-                if (destinationFile == nullptr) {
-                    fclose(sourceFile);
-                    closedir(dir);
-                    return false;
-                }
-
-                int c;
-                while ((c = fgetc(sourceFile)) != EOF) {
-                    fputc(c, destinationFile);
-                }
-
-                fclose(sourceFile);
-                fclose(destinationFile);
-            }
-        }
+      FILE* destinationFile = fopen(destinationPath, "w");
+      if (destinationFile == nullptr) {
+        fclose(sourceFile);
+        closedir(dir);
+        return false;
+      }
+
+      int c;
+      while ((c = fgetc(sourceFile)) != EOF) {
+        fputc(c, destinationFile);
+      }
+
+      fclose(sourceFile);
+      fclose(destinationFile);
     }
+  }
 
-    closedir(dir);
-    return true;
+  closedir(dir);
+  return true;
 }
 
 [[gnu::unused]] static std::string replaceLastPathComponent(const std::string& path, const std::string& newComponent) {