fix clang tidy et format

epitar/src/main.c

@@ -3,6 +3,52 @@
 #include "utils/config.h"
 #include "utils/errors.h"
 
+static int handle_err_code(enum error_code err, struct config *config)
+{
+    int ret_code = 0;
+    switch (err)
+    {
+    case SUCCESS:
+        ret_code = 0;
+        break;
+
+        // case UNKNOWN_ERROR:
+        //     printf("epitar: error extracting tarball %s\n",
+        //     config.archive_file); ret_code = 3; break;
+
+    case BAD_CHECKSUM:
+        puts("epitar: bad checksum");
+        ret_code = 2;
+        break;
+
+    case NOT_IMPLEMENTED:
+        puts("epitar: Function not implemented");
+        ret_code = 2;
+        break;
+
+    case FILE_NOT_FOUND:
+        if (config->mode == ARCHIVE)
+        {
+            // Rest of code should have printed the first part of the line
+            printf(" to tarball %s\n", config->archive_file);
+            ret_code = 3;
+            break;
+        }
+        __attribute__((fallthrough));
+
+    case EMPTY_ARCHIVE:
+        puts("epitar: cowardly refusing to create an empty archive");
+        ret_code = 1;
+        break;
+
+    default:
+        printf("epitar: error extracting tarball %s\n", config->archive_file);
+        ret_code = 3;
+        break;
+    }
+    return ret_code;
+}
+
 int main(int argc, char **argv)
 {
     // Handle args
@@ -32,48 +78,7 @@ int main(int argc, char **argv)
         err = archive(config.archive_file, config.input_files, config.verbose);
     }
 
-    int ret_code = 0;
+    int ret_code = handle_err_code(err, &config);
-    switch (err)
-    {
-    case SUCCESS:
-        ret_code = 0;
-        break;
-
-        // case UNKNOWN_ERROR:
-        //     printf("epitar: error extracting tarball %s\n",
-        //     config.archive_file); ret_code = 3; break;
-
-    case BAD_CHECKSUM:
-        puts("epitar: bad checksum");
-        ret_code = 2;
-        break;
-
-    case NOT_IMPLEMENTED:
-        puts("epitar: Function not implemented");
-        ret_code = 2;
-        break;
-
-    case FILE_NOT_FOUND:
-        if (config.mode == ARCHIVE)
-        {
-            // Rest of code should have printed the first part of the line
-            printf(" to tarball %s\n", config.archive_file);
-            ret_code = 3;
-            break;
-        }
-        __attribute__((fallthrough));
-
-    case EMPTY_ARCHIVE:
-        puts("epitar: cowardly refusing to create an empty archive");
-        ret_code = 1;
-        break;
-
-    default:
-        printf("epitar: error extracting tarball %s\n", config.archive_file);
-        ret_code = 3;
-        break;
-    }
-
     destroy_config(&config);
     return ret_code;
 }

epitar/src/tar/archive.h

@@ -1,10 +1,10 @@
 #ifndef ARCHIVE_H
 #define ARCHIVE_H
 
-#include "../utils/errors.h"
-
 #include <stdbool.h>
 
+#include "../utils/errors.h"
+
 /* @brief Archives the listed `files` into `archive_name`
  * @return 0 on success, the corresponding error code otherwise
  */

epitar/src/tar/extract.c

@@ -49,9 +49,14 @@ static enum error_code extract_ustar(FILE *stream, struct ustar_header *header)
 static enum error_code extract_unixtar(FILE *stream,
                                        struct unixtar_header *header)
 {
-    (void)stream;
+    struct file_stats stats = { .name = header->name,
-    (void)header;
+                                .length = strtol(header->size, NULL, 8),
-    return NOT_IMPLEMENTED;
+                                .content_stream = stream,
+                                .mode = header->mode,
+                                .uid = header->uid,
+                                .gid = header->gid,
+                                .mtime = header->mtime };
+    return create_file(&stats);
 }
 
 /**

epitar/src/utils/filesystem.c

@@ -180,7 +180,7 @@ char *listdirectory(char *path)
     return de->d_name;
 }
 
-void freedirectory()
+void freedirectory(void)
 {
     if (current_dir != NULL)
     {

epitar/src/utils/filesystem.h

@@ -59,6 +59,6 @@ char *listdirectory(char *path);
  * related allocated memory
  * @warn Use this function if you only partially listed a directory previously.
  */
-void freedirectory();
+void freedirectory(void);
 
 #endif // FILESYSTEM_H

epitar/src/utils/misc.c

@@ -1,3 +0,0 @@
-#include "utils.h"
-
-#include <stdio.h>

epitar/src/utils/misc.h

@@ -1,6 +0,0 @@
-#ifndef MISC_H
-#define MISC_H
-
-void print_str_array(const char **input_files);
-
-#endif /* MISC_H */