Ensure pack is initialised when loading an asset pack

.gitignore

@@ -1,5 +1,5 @@
 .cache
 compile_commands.json
 pckr
-pakrd.a
+libpakrd.so
 dist

.gitmodules

@@ -0,0 +1,8 @@
+[submodule "intern/c-cpp-aliases"]
+	path = intern/c-cpp-aliases
+	url = https://git.thewizardapprentice.com/abdelrahman/c-cpp-aliases.git
+	branch = main
+[submodule "intern/cpath"]
+	path = intern/cpath
+	url = https://git.thewizardapprentice.com/abdelrahman/cpath.git
+	branch = main

compile

@@ -1,31 +1,10 @@
 #!/bin/bash
 
-#ESCAPE SEQUENCES
-CLEAR="\033[0m"
-BOLD="\033[1"
-FG_BLACK="\033[30"
-FG_RED="\033[31"
-FG_GREEN="\033[32"
-FG_YELLOW="\033[33"
-FG_BLUE="\033[34"
-FG_MAGENTA="\033[35"
-FG_CYAN="\033[36"
-FG_WHITE="\033[37"
-FG_BR_BLACK="\033[90"
-FG_BR_RED="\033[91"
-FG_BR_GREEN="\033[92"
-FG_BR_YELLOW="\033[93"
-FG_BR_BLUE="\033[94"
-FG_BR_MAGENTA="\033[95"
-FG_BR_CYAN="\033[96"
-FG_BR_WHITE="\033[97"
-
-CC=clang
-AR=ar
-CFLAGS="-Wall -Werror -pedantic -Iinclude "
+CC=gcc
+CFLAGS="-Wall -Werror -pedantic -Iinclude -Iintern/c-cpp-aliases -Iintern/cpath/include "
 
 PCKR_SRC="\
-	src/path_utils.c \
+	intern/cpath/src/cpath.c \
 	src/argparse.c \
 	src/io.c \
 	src/darr.c \
@@ -34,20 +13,20 @@ PCKR_SRC="\
 	src/main.c"
 
 PAKRD_SRC="\
-	src/path_utils.c \
+	intern/cpath/src/cpath.c \
 	src/io.c \
 	src/darr.c \
 	src/pak.c"
-PAKRD_FLAGS="-c -fPIC"
+PAKRD_FLAGS="-fPIC -rdynamic -shared "
 
 PCKR_TEST_SRC="\
-	src/path_utils.c \
+	intern/cpath/src/cpath.c \
 	src/io.c \
 	src/pckr_test.c"
 PCKR_TEST_OUT="pckr_test"
 
 PAKRD_TEST_SRC="\
-	src/path_utils.c \
+	intern/cpath/src/cpath.c \
 	src/io.c \
 	src/darr.c \
 	src/pak.c \
@@ -74,13 +53,13 @@ done
 if [[ $BUILD_TYPE == "debug" ]]; then
 	CFLAGS+="-g "
 	PCKR_OUT="pckr"
-	PAKRD_OUT="pakrd.a"
+	PAKRD_OUT="libpakrd.so"
 else
-	mkdir dist
+	mkdir -p dist
 	cp ./include/pak.h dist
 	CFLAGS+="-O3 "
 	PCKR_OUT="dist/pckr"
-	PAKRD_OUT="dist/pakrd.a"
+	PAKRD_OUT="dist/libpakrd.so"
 fi
 
 
@@ -88,9 +67,8 @@ fi
 (set -x ; $CC $CFLAGS $PCKR_SRC -o $PCKR_OUT)
 
 
-# Build pakrd static library
-(set -x ; $CC $CFLAGS $PAKRD_FLAGS $PAKRD_SRC ; $AR r $PAKRD_OUT *.o)
-rm *.o
+# Build pakrd shared library
+(set -x ; $CC $CFLAGS $PAKRD_FLAGS $PAKRD_SRC -o $PAKRD_OUT)
 
 
 # Build pckr_test executable
@@ -105,6 +83,27 @@ rm *.o
 ./pckr ./test_assets ./assets
 
 
+#ESCAPE SEQUENCES
+CLEAR="\033[0m"
+BOLD="\033[1"
+FG_BLACK="\033[30"
+FG_RED="\033[31"
+FG_GREEN="\033[32"
+FG_YELLOW="\033[33"
+FG_BLUE="\033[34"
+FG_MAGENTA="\033[35"
+FG_CYAN="\033[36"
+FG_WHITE="\033[37"
+FG_BR_BLACK="\033[90"
+FG_BR_RED="\033[91"
+FG_BR_GREEN="\033[92"
+FG_BR_YELLOW="\033[93"
+FG_BR_BLUE="\033[94"
+FG_BR_MAGENTA="\033[95"
+FG_BR_CYAN="\033[96"
+FG_BR_WHITE="\033[97"
+
+
 # Run pckr_test
 (./$PCKR_TEST_OUT && echo -e "\npckr_test  [${FG_BR_GREEN}m${BOLD}m SUCCESS $CLEAR]") || echo -e "\npckr_test  [${FG_BR_RED}m${BOLD}m FAILURE $CLEAR]"
 

include/aliases.h

@@ -1,27 +0,0 @@
-#ifndef ALIASES_H
-#define ALIASES_H
-
-#include <stdint.h>
-
-#define u8 uint8_t
-#define u16 uint16_t
-#define u32 uint32_t
-#define u64 uint64_t
-
-#define i8 int8_t
-#define i16 int16_t
-#define i32 int32_t
-#define i64 int64_t
-
-#define f32 float
-#define f64 double
-#define f128 long double
-
-#define INTERNAL static
-#define PERSISTENT static
-
-#ifdef __cplusplus
-#define CLASS_MEMBER static
-#endif // __cplusplus
-
-#endif // !ALIASES_H

include/argparse.h

@@ -1,6 +1,10 @@
 #ifndef ARGPARSE_H
 #define ARGPARSE_H
 
+#ifdef __cplusplus
+extern "C" {
+#endif // __cplusplus
+
 #include "aliases.h"
 #include <linux/limits.h>
 #include <stdbool.h>
@@ -17,4 +21,8 @@ typedef struct {
 
 argparser_t parse_args(i32 argc, char *argv[]);
 
+#ifdef __cplusplus
+}
+#endif // __cplusplus
+
 #endif // !ARGPARSE_H

include/darr.h

@@ -1,6 +1,10 @@
 #ifndef D_ARR_H
 #define D_ARR_H
 
+#ifdef __cplusplus
+extern "C" {
+#endif // __cplusplus
+
 #include "aliases.h"
 #include <stdbool.h>
 
@@ -13,4 +17,8 @@ i64 darr_add(darr_t **darr, void *item);
 void **darr_get_items(darr_t *darr);
 void darr_free(darr_t **darr);
 
+#ifdef __cplusplus
+}
+#endif // __cplusplus
+
 #endif // !D_ARR_H

include/io.h

@@ -1,6 +1,10 @@
 #ifndef IO_H
 #define IO_H
 
+#ifdef __cplusplus
+extern "C" {
+#endif // __cplusplus
+
 #include "aliases.h"
 #include <linux/limits.h>
 #include <stdbool.h>
@@ -21,4 +25,8 @@ u64 get_file_length(FILE *fp);
 void read_entire_file(void *dst, FILE *fp);
 dirwalk_result_t walk_dir(dirwalk_t **dirwalk, const char *filepath);
 
+#ifdef __cplusplus
+}
+#endif // __cplusplus
+
 #endif // !IO_H

include/pak.h

@@ -1,6 +1,10 @@
 #ifndef PAK_H
 #define PAK_H
 
+#ifdef __cplusplus
+extern "C" {
+#endif // __cplusplus
+
 #include "aliases.h"
 #include <stdbool.h>
 
@@ -45,4 +49,8 @@ buf_t *read_file_from_pack(asset_pack_t *pack, const char *filename);
 void close_pack_file(buf_t **buf);
 void close_asset_pack(asset_pack_t **pack);
 
+#ifdef __cplusplus
+}
+#endif // __cplusplus
+
 #endif // !PAK_H

include/path_utils.h

@@ -1,18 +0,0 @@
-#ifndef PATH_UTILS_H
-#define PATH_UTILS_H
-
-#include "aliases.h"
-
-#define NUMPARTS(...)                                                          \
-  (sizeof((const char *[]){"", __VA_ARGS__}) / sizeof(const char *) - 1)
-
-#define JOIN_PATH(DST, ...) join_path(DST, NUMPARTS(__VA_ARGS__), __VA_ARGS__)
-
-void join_path(char *dst, u64 count, ...);
-
-#define DIRNAME(DST, PATH) dirup(DST, 1, PATH)
-#define DIRUP(DST, COUNT, PATH) dirup(DST, COUNT, PATH)
-
-void dirup(char *dst, u64 levels, const char *path);
-
-#endif // !PATH_UTILS_H

include/pckr.h

@@ -1,8 +1,16 @@
 #ifndef PCKR_H
 #define PCKR_H
 
+#ifdef __cplusplus
+extern "C" {
+#endif // __cplusplus
+
 #include "aliases.h"
 
 i32 run_pckr(i32 argc, char *argv[]);
 
+#ifdef __cplusplus
+}
+#endif // __cplusplus
+
 #endif // !PCKR_H

include/test_utils.h

@@ -1,6 +1,10 @@
 #ifndef TEST_UTILS_H
 #define TEST_UTILS_H
 
+#ifdef __cplusplus
+extern "C" {
+#endif // __cplusplus
+
 #define ARRLEN(ARR) (sizeof ARR / sizeof ARR[0])
 
 typedef struct {
@@ -8,4 +12,8 @@ typedef struct {
   const char *contents;
 } asset_t;
 
+#ifdef __cplusplus
+}
+#endif // __cplusplus
+
 #endif // !TEST_UTILS_H

src/argparse.c

@@ -7,10 +7,9 @@ argparser_t parse_args(i32 argc, char *argv[]) {
     return (argparser_t){0};
   }
 
-  argparser_t output = {
-      .succeeded = true,
-      .args = (pckr_args_t){0},
-  };
+  argparser_t output = {0};
+  output.succeeded = true;
+  output.args = (pckr_args_t){0};
 
   realpath(argv[1], output.args.dir);
   realpath(argv[2], output.args.output);

src/darr.c

@@ -40,7 +40,7 @@ i64 darr_add(darr_t **darr, void *item) {
       return INVALID_IDX;
     }
 
-    (*darr)->capacity = orig->capacity + BASE_CAPACITY;
+    (*darr)->capacity += BASE_CAPACITY;
   }
 
   u64 index = (*darr)->count;

src/io.c

@@ -1,6 +1,6 @@
 #include "io.h"
 #include "aliases.h"
-#include "path_utils.h"
+#include "cpath.h"
 #include <dirent.h>
 #include <linux/limits.h>
 #include <stdio.h>
@@ -16,8 +16,8 @@ struct dirwalk {
   u64 count;
   FILE *current_file;
   DIR *current_dir;
-  char dst[PATH_MAX];
-  DIR *directories[MAX_DIR_DEPTH];
+  char dst[PATH_MAX + 1];
+  DIR *directories[MAX_DIR_DEPTH + 1];
 };
 
 u64 get_file_length(FILE *fp) {
@@ -73,7 +73,7 @@ dirwalk_result_t walk_dir(dirwalk_t **dirwalk, const char *filepath) {
 
   dirwalk_t *dw = *dirwalk;
 
-  char tmp[PATH_MAX] = {0};
+  char tmp[PATH_MAX + 1] = {0};
 
   dirent_t *ep = NULL;
   stat_t dirstat = {0};
@@ -102,7 +102,7 @@ dirwalk_result_t walk_dir(dirwalk_t **dirwalk, const char *filepath) {
 
       result.reading = true;
       result.fp = dw->current_file;
-      strncpy(result.name, ep->d_name, NAME_MAX);
+      strncpy(result.name, ep->d_name, NAME_MAX + 1);
 
       return result;
     }
@@ -116,7 +116,7 @@ dirwalk_result_t walk_dir(dirwalk_t **dirwalk, const char *filepath) {
       if ((dirstat.st_mode & S_IFDIR) == S_IFDIR) {
         memset(dw->dst, 0, length);
 
-        strncpy(dw->dst, tmp, length);
+        strncpy(dw->dst, tmp, length + 1);
 
         dw->directories[(dw->count)++] = opendir(dw->dst);
 
@@ -128,7 +128,7 @@ dirwalk_result_t walk_dir(dirwalk_t **dirwalk, const char *filepath) {
 
         result.reading = true;
         result.fp = dw->current_file;
-        strncpy(result.name, ep->d_name, NAME_MAX);
+        strncpy(result.name, ep->d_name, NAME_MAX + 1);
 
         return result;
       }
@@ -150,7 +150,7 @@ dirwalk_result_t walk_dir(dirwalk_t **dirwalk, const char *filepath) {
 
     u64 length = strlen(dw->dst);
 
-    strncpy(tmp, dw->dst, length);
+    strcpy(tmp, dw->dst);
 
     memset(dw->dst, 0, length);
 

src/pak.c

@@ -34,13 +34,11 @@ void write_pak(const pak_t *pak, toc_entry_t **toc_entries,
 bool create_asset_pack(const char *dirpath, const char *output) {
   bool created = false;
 
-  char dst[PATH_MAX] = {0};
+  char dst[PATH_MAX + 1] = {0};
 
-  u64 output_length = strlen(output);
-  strncpy(dst, output, output_length);
+  strcpy(dst, output);
 
-  u64 ext_length = strlen(PAK_EXT);
-  strncat(dst, PAK_EXT, ext_length);
+  strcat(dst, PAK_EXT);
 
   FILE *fp = fopen(dst, "wb");
   if (!fp) {
@@ -130,6 +128,8 @@ asset_pack_t *load_asset_pack(const char *filepath) {
     return NULL;
   }
 
+  memset((void *)pack, 0, sizeof(asset_pack_t));
+
   pack->fp = fopen(full_path, "rb");
   if (!(pack->fp)) {
     close_asset_pack(&pack);
@@ -229,7 +229,9 @@ void close_asset_pack(asset_pack_t **pack) {
     free((*pack)->toc);
   }
 
-  fclose((*pack)->fp);
+  if ((*pack)->fp) {
+    fclose((*pack)->fp);
+  }
 
   free(*pack);
 
@@ -250,7 +252,7 @@ toc_entry_t *create_toc_entry(const char *name, u64 offset) {
   entry->offset = offset;
   entry->length = length + 1;
 
-  strncpy(entry->name, name, length);
+  strcpy(entry->name, name);
 
   return entry;
 }

src/path_utils.c

@@ -1,76 +0,0 @@
-#include "path_utils.h"
-#include "aliases.h"
-#include <stdarg.h>
-#include <stdio.h>
-#include <string.h>
-
-#if defined(__unix__) || defined(__APPLE__) || defined(__ANDROID__)
-INTERNAL char path_sep = '/';
-#elif defined(_WIN32) || defined(_WIN64)
-INTERNAL char path_sep = '\';
-#endif
-
-void join_root_and_leaf(const char *root, const char *leaf, char *dst);
-
-void join_path(char *dst, u64 count, ...) {
-  va_list args;
-
-  va_start(args, count);
-
-  for (u64 i = 0; i < count; ++i) {
-    join_root_and_leaf(dst, va_arg(args, const char *), dst);
-  }
-
-  va_end(args);
-}
-
-void dirup(char *dst, u64 levels, const char *path) {
-  if (levels < 1) {
-    return;
-  }
-
-  u64 end_index = 0;
-  u64 sep_count = 0;
-
-  u64 length = strlen(path);
-  if (path[length - 1] == path_sep) {
-    --length;
-  }
-
-  for (u64 i = length - 1; i >= 0; --i) {
-    if (path[i] == path_sep) {
-      ++sep_count;
-      end_index = i;
-
-      if (sep_count == levels) {
-        break;
-      }
-    }
-  }
-
-  if (sep_count < levels) {
-    end_index = 0;
-  }
-
-  strncpy(dst, path, end_index);
-}
-
-void join_root_and_leaf(const char *root, const char *leaf, char *dst) {
-  u64 root_length = strlen(root);
-  u64 root_end = root_length - 1;
-
-  u64 leaf_length = strlen(leaf);
-  u64 leaf_start = 0;
-
-  if (root[root_end] == path_sep) {
-    --root_end;
-  }
-
-  if (leaf[leaf_start] == path_sep) {
-    ++leaf_start;
-  }
-
-  strncpy(dst, root, ++root_end);
-  dst[root_end] = path_sep;
-  strncpy(&(dst[++root_end]), &(leaf[leaf_start]), leaf_length - leaf_start);
-}