Add table iteration API and improve parsing robustness

CMakeLists.txt

@@ -53,6 +53,8 @@ target_link_libraries(${PROJECT_NAME} PRIVATE ${PROJECT_OBJECTS})
 add_compile_definitions(_XOPEN_SOURCE=700)
 
 
+# CTest-Unterstützung aktivieren
+enable_testing()
 
 # --- Add tools ---
 add_subdirectory(tools)

src/common/iniparser.c

@@ -142,7 +142,7 @@ static bool is_digit(char ch) {
 }
 
 static bool is_key_char(char ch) {
-    return is_letter(ch) || is_digit(ch) || ch == '.' || ch == '-' || ch == '_';
+    return is_letter(ch) || is_digit(ch) || ch == '.' || ch == '-' || ch == '_' || ch == ':';
 }
 
 static bool is_at_end(const IniParser *parser) {

src/common/iniparser.h

@@ -57,7 +57,7 @@
 <WS>            ::= ' ' | '\t'
 
 <LETTER>        ::= [a-zA-Z]
-<KEY_CHAR>      ::= <LETTER> | <DIGIT> | '.' | '-' | '_'
+<KEY_CHAR>      ::= <LETTER> | <DIGIT> | '.' | '-' | '_' | ':'
 <KEY>           ::= <KEY_CHAR> <KEY_CHAR>*
 
 *****************************************/

src/common/string.c

@@ -316,6 +316,9 @@ void String_Set(String *str, String src) {
 }
 
 void String_Take(String *dst, String *src) {
+    if (!dst || ! src) {
+        return;
+    }
     String_Deinit(dst);
     *dst = String_TakeCStr(src->bytes);
 

src/common/string.h

@@ -178,7 +178,7 @@ void String_Set(String *str, String src);
  * 
  * Its for situation where dst is already initialized and your want to fill it
  * with the content of another string without moving memory around.
- * dst will be deinitialized!
+ * src will be deinitialized!
  */
 void String_Take(String *dst, String *src);
 

src/common/table.c

@@ -220,6 +220,9 @@ void *Table_Get(const Table *table, const char *key) {
     if (!table) {
         logFatal("Invalid table in Table_Get().");
     }
+    if (table->used == 0) {
+        return NULL;
+    }
     if (!key) {
         logWarn("Table_Get() was called with key == NULL.");
         return NULL;
@@ -236,6 +239,9 @@ void Table_Delete(Table *table, const char *key) {
     if (!table) {
         logFatal("Invalid table in Table_Delete().");
     }
+    if (table->used == 0) {
+        return;
+    }
     if (!key) {
         logWarn("Table_Delete() is called with key == NULL.");
         return;
@@ -255,6 +261,9 @@ bool Table_Has(const Table *table, const char *key) {
     if (!table) {
         logFatal("Invalid table in Table_Has().");
     }
+    if (table->used == 0) {
+        return false;
+    }
     if (!key) {
         logWarn("Table_Has() was called with key == NULL.");
         return false;
@@ -267,10 +276,20 @@ bool Table_HasOwnership(const Table *table, const char *key) {
     if (!table) {
         logFatal("Invalid table in Table_HasOwnership().");
     }
+    if (table->used == 0) {
+        return false;
+    }
     if (!key) {
         logWarn("Table_HasOwnership() was called with key == NULL.");
         return false;
     }
     TableSlot *slot = find_slot(table, key);
     return (slot->state == TABLE_SLOT_USED && slot->destructor != NULL);
 }
+
+size_t Table_GetUsage(const Table *table) {
+    if (!table) {
+        return 0;
+    }
+    return table->used;
+}

src/common/table.h

@@ -97,5 +97,9 @@ bool Table_Has(const Table *table, const char *key);
  */
 bool Table_HasOwnership(const Table *table, const char *key);
 
+/**
+ * @brief Return the current number of non-free table slots.
+ */
+size_t Table_GetUsage(const Table *table);
 
 #endif

src/common/tableiterator.c

@@ -0,0 +1,24 @@
+#include "tableiterator.h"
+
+TableIterator TableIterator_Begin(const Table *table) {
+    return (TableIterator){
+        .table = table,
+        .index = -1,
+        .current = NULL
+    };
+}
+
+bool TableIterator_Next(TableIterator *it) {
+    if (!it || !it->table) {
+        return false;
+    }
+    for (size_t i = (size_t)(it->index + 1); i < it->table->capacity; i++) {
+        TableSlot *slot = &it->table->slots[i];
+        if (slot->state == TABLE_SLOT_USED) {
+            it->index = i;
+            it->current = slot;
+            return true;
+        }
+    }
+    return false;
+}

src/common/tableiterator.h

@@ -0,0 +1,17 @@
+#ifndef TABLEITERATOR_H
+#define TABLEITERATOR_H
+
+#include <sys/types.h>
+#include <stdbool.h>
+#include "table.h"
+
+typedef struct _TableIterator {
+    const Table *table;
+    ssize_t index;
+    const TableSlot *current;
+} TableIterator;
+
+TableIterator TableIterator_Begin(const Table *table);
+bool TableIterator_Next(TableIterator *it);
+
+#endif