[Refacto] change public api funtion to use a pointer to a argus object

Author: lucocozz

docs/docs/advanced/custom-handlers.md

@@ -116,7 +116,7 @@ int main(int argc, char **argv)
     argus_parse(&argus, argc, argv);
     
     // Access custom type
-    endpoint_t *ep = (endpoint_t*)argus_get(argus, "endpoint").as_ptr;
+    endpoint_t *ep = (endpoint_t*)argus_get(&argus, "endpoint").as_ptr;
     if (ep)
         printf("Connecting to %s:%d\n", ep->host, ep->port);
 

docs/docs/advanced/custom-validators.md

@@ -164,7 +164,7 @@ int max_greater_than_min_validator(argus_t *argus, void *option_ptr, validator_d
     const char *min_option = (const char *)data.custom;
 
     int max_value = option->value.as_int;
-    int min_value = argus_get(*argus, min_option).as_int;
+    int min_value = argus_get(argus, min_option).as_int;
 
     if (max_value <= min_value) {
         ARGUS_PARSING_ERROR(argus, "Max value %d must be greater than min value %d",

docs/docs/api-reference/overview.md

@@ -265,9 +265,9 @@ int main(int argc, char **argv)
     if (argus_parse(&argus, argc, argv) != ARGUS_SUCCESS)
         return 1;
 
-    bool verbose = argus_get(argus, "verbose").as_bool;
-    const char *output = argus_get(argus, "output").as_string;
-    const char *input = argus_get(argus, "input").as_string;
+    bool verbose = argus_get(&argus, "verbose").as_bool;
+    const char *output = argus_get(&argus, "output").as_string;
+    const char *input = argus_get(&argus, "input").as_string;
 
     // Application logic here
 
@@ -281,8 +281,8 @@ int main(int argc, char **argv)
 ```c
 int add_command(argus_t *argus, void *data)
 {
-    const char *file = argus_get(*argus, "file").as_string;
-    bool force = argus_get(*argus, "force").as_bool;
+    const char *file = argus_get(argus, "file").as_string;
+    bool force = argus_get(argus, "force").as_bool;
     // Command logic here
     return 0;
 }
@@ -306,10 +306,9 @@ int main(int argc, char **argv)
     if (argus_parse(&argus, argc, argv) != ARGUS_SUCCESS)
         return 1;
     
-    if (argus_has_command(argus)) {
+    if (argus_has_command(&argus))
         return argus_exec(&argus, NULL);
-    }
-    
+
     argus_free(&argus);
     return 0;
 }

docs/docs/appendices/cheat-sheet.md

@@ -20,7 +20,7 @@ int main(int argc, char **argv)
         return 1;
 
     // Access values
-    bool flag = argus_get(argus, "flag").as_bool;
+    bool flag = argus_get(&argus, "flag").as_bool;
 
     argus_free(&argus);
     return 0;
@@ -132,23 +132,23 @@ FLAGS(FLAG_SORTED_VALUE)    // Sort map by values
 
 ```c
 // Basic access
-bool verbose = argus_get(argus, "verbose").as_bool;
-const char *output = argus_get(argus, "output").as_string;
+bool verbose = argus_get(&argus, "verbose").as_bool;
+const char *output = argus_get(&argus, "output").as_string;
 
 // Check if set
-if (argus_is_set(argus, "output")) {
+if (argus_is_set(&argus, "output")) {
     // User provided this option
 }
 
 // Array access
-size_t count = argus_count(argus, "tags");
-const char *first = argus_array_get(argus, "tags", 0).as_string;
+size_t count = argus_count(&argus, "tags");
+const char *first = argus_array_get(&argus, "tags", 0).as_string;
 
 // Map access
-const char *user = argus_map_get(argus, "env", "USER").as_string;
+const char *user = argus_map_get(&argus, "env", "USER").as_string;
 
 // Iterators
-argus_array_it_t it = argus_array_it(argus, "tags");
+argus_array_it_t it = argus_array_it(&argus, "tags");
 while (argus_array_next(&it)) {
     printf("%s\n", it.value.as_string);
 }
@@ -159,8 +159,8 @@ while (argus_array_next(&it)) {
 ```c
 int add_action(argus_t *argus, void *data)
 {
-    const char *file = argus_get(*argus, "file").as_string;     // Local option
-    bool verbose = argus_get(*argus, ".verbose").as_bool;       // Global option
+    const char *file = argus_get(argus, "file").as_string;     // Local option
+    bool verbose = argus_get(argus, ".verbose").as_bool;       // Global option
     return 0;
 }
 
@@ -178,7 +178,7 @@ ARGUS_OPTIONS(options,
 int main(int argc, char **argv)
 {
     // ... parse ...
-    if (argus_has_command(argus)) {
+    if (argus_has_command(&argus)) {
         return argus_exec(&argus, NULL);
     }
 }
@@ -245,7 +245,7 @@ OPTION_BASE('e', "endpoint", VALUE_TYPE_CUSTOM,
             HANDLER(endpoint_handler), FREE_HANDLER(endpoint_free))
 
 // Access
-endpoint_t *ep = (endpoint_t*)argus_get(argus, "endpoint").as_ptr;
+endpoint_t *ep = (endpoint_t*)argus_get(&argus, "endpoint").as_ptr;
 ```
 
 ## Groups

docs/docs/appendices/contributing.md

@@ -209,7 +209,7 @@ void test_option_parsing(void)
     char *argv[] = {"test", "--output", "file.txt"};
     assert(argus_parse(&argus, 3, argv) == ARGUS_SUCCESS);
 
-    const char *output = argus_get(argus, "output").as_string;
+    const char *output = argus_get(&argus, "output").as_string;
     assert(strcmp(output, "file.txt") == 0);
 
     argus_free(&argus);

docs/docs/examples/configuration-tool.md

@@ -48,46 +48,42 @@ ARGUS_OPTIONS(
 
 void print_array_strings(argus_t argus, const char *name)
 {
-    if (!argus_is_set(argus, name)) return;
+    if (!argus_is_set(&argus, name)) return;
 
     printf("%s:\n", name);
-    argus_array_it_t it = argus_array_it(argus, name);
-    while (argus_array_next(&it)) {
+    argus_array_it_t it = argus_array_it(&argus, name);
+    while (argus_array_next(&it))
         printf("  - %s\n", it.value.as_string);
-    }
 }
 
 void print_array_ints(argus_t argus, const char *name)
 {
-    if (!argus_is_set(argus, name)) return;
+    if (!argus_is_set(&argus, name)) return;
 
     printf("%s:\n", name);
-    argus_array_it_t it = argus_array_it(argus, name);
-    while (argus_array_next(&it)) {
+    argus_array_it_t it = argus_array_it(&argus, name);
+    while (argus_array_next(&it))
         printf("  - %d\n", it.value.as_int);
-    }
 }
 
 void print_map_strings(argus_t argus, const char *name)
 {
-    if (!argus_is_set(argus, name)) return;
+    if (!argus_is_set(&argus, name)) return;
 
     printf("%s:\n", name);
-    argus_map_it_t it = argus_map_it(argus, name);
-    while (argus_map_next(&it)) {
+    argus_map_it_t it = argus_map_it(&argus, name);
+    while (argus_map_next(&it))
         printf("  %s = %s\n", it.key, it.value.as_string);
-    }
 }
 
 void print_map_bools(argus_t argus, const char *name)
 {
-    if (!argus_is_set(argus, name)) return;
+    if (!argus_is_set(&argus, name)) return;
 
     printf("%s:\n", name);
-    argus_map_it_t it = argus_map_it(argus, name);
-    while (argus_map_next(&it)) {
+    argus_map_it_t it = argus_map_it(&argus, name);
+    while (argus_map_next(&it))
         printf("  %s = %s\n", it.key, it.value.as_bool ? "true" : "false");
-    }
 }
 
 int main(int argc, char **argv)
@@ -100,17 +96,16 @@ int main(int argc, char **argv)
         return 1;
 
     // Basic values
-    const char *name = argus_get(argus, "name").as_string;
-    const char *host = argus_get(argus, "host").as_string;
-    int port = argus_get(argus, "port").as_int;
+    const char *name = argus_get(&argus, "name").as_string;
+    const char *host = argus_get(&argus, "host").as_string;
+    int port = argus_get(&argus, "port").as_int;
 
     printf("=== Service Configuration ===\n");
     printf("Name: %s\n", name);
     printf("Host: %s:%d\n", host, port);
 
-    if (argus_is_set(argus, "contact")) {
-        printf("Contact: %s\n", argus_get(argus, "contact").as_string);
-    }
+    if (argus_is_set(&argus, "contact"))
+        printf("Contact: %s\n", argus_get(&argus, "contact").as_string);
     printf("\n");
 
     // Collections
@@ -121,8 +116,8 @@ int main(int argc, char **argv)
 
     // Generate config file
     const char *output = "config.json";
-    if (argus_is_set(argus, "output")) {
-        output = argus_get(argus, "output").as_string;
+    if (argus_is_set(&argus, "output")) {
+        output = argus_get(&argus, "output").as_string;
     }
 
     FILE *config = fopen(output, "w");
@@ -136,14 +131,14 @@ int main(int argc, char **argv)
     fprintf(config, "  \"host\": \"%s\",\n", host);
     fprintf(config, "  \"port\": %d", port);
 
-    if (argus_is_set(argus, "contact")) {
-        fprintf(config, ",\n  \"contact\": \"%s\"", argus_get(argus, "contact").as_string);
+    if (argus_is_set(&argus, "contact")) {
+        fprintf(config, ",\n  \"contact\": \"%s\"", argus_get(&argus, "contact").as_string);
     }
 
     // Add arrays
-    if (argus_is_set(argus, "tags")) {
+    if (argus_is_set(&argus, "tags")) {
         fprintf(config, ",\n  \"tags\": [");
-        argus_array_it_t it = argus_array_it(argus, "tags");
+        argus_array_it_t it = argus_array_it(&argus, "tags");
         bool first = true;
         while (argus_array_next(&it)) {
             if (!first) fprintf(config, ", ");
@@ -153,9 +148,9 @@ int main(int argc, char **argv)
         fprintf(config, "]");
     }
 
-    if (argus_is_set(argus, "ports")) {
+    if (argus_is_set(&argus, "ports")) {
         fprintf(config, ",\n  \"ports\": [");
-        argus_array_it_t it = argus_array_it(argus, "ports");
+        argus_array_it_t it = argus_array_it(&argus, "ports");
         bool first = true;
         while (argus_array_next(&it)) {
             if (!first) fprintf(config, ", ");
@@ -166,9 +161,9 @@ int main(int argc, char **argv)
     }
 
     // Add maps
-    if (argus_is_set(argus, "env")) {
+    if (argus_is_set(&argus, "env")) {
         fprintf(config, ",\n  \"environment\": {");
-        argus_map_it_t it = argus_map_it(argus, "env");
+        argus_map_it_t it = argus_map_it(&argus, "env");
         bool first = true;
         while (argus_map_next(&it)) {
             if (!first) fprintf(config, ", ");
@@ -179,9 +174,9 @@ int main(int argc, char **argv)
         fprintf(config, "}");
     }
 
-    if (argus_is_set(argus, "features")) {
+    if (argus_is_set(&argus, "features")) {
         fprintf(config, ",\n  \"features\": {");
-        argus_map_it_t it = argus_map_it(argus, "features");
+        argus_map_it_t it = argus_map_it(&argus, "features");
         bool first = true;
         while (argus_map_next(&it)) {
             if (!first) fprintf(config, ", ");