jshn: properly support JSON "null" type
authorJo-Philipp Wich <jo@mein.io>
Sun, 7 Jan 2018 14:46:31 +0000 (15:46 +0100)
committerJo-Philipp Wich <jo@mein.io>
Sun, 7 Jan 2018 15:02:02 +0000 (16:02 +0100)
Instead of abort parsing, properly deal with "null" values by implementing
support for reading and formatting such values.

Signed-off-by: Jo-Philipp Wich <jo@mein.io>
(cherry picked from commit 1c08e80313fd487112c48346889cc57badeef751)

jshn.c
sh/jshn.sh

diff --git a/jshn.c b/jshn.c
index 79136dd..3188af5 100644 (file)
--- a/jshn.c
+++ b/jshn.c
@@ -105,9 +105,6 @@ static int add_json_element(const char *key, json_object *obj)
 {
        char *type;
 
-       if (!obj)
-               return -1;
-
        switch (json_object_get_type(obj)) {
        case json_type_object:
                type = "object";
@@ -127,6 +124,9 @@ static int add_json_element(const char *key, json_object *obj)
        case json_type_double:
                type = "double";
                break;
+       case json_type_null:
+               type = "null";
+               break;
        default:
                return -1;
        }
@@ -159,6 +159,9 @@ static int add_json_element(const char *key, json_object *obj)
        case json_type_double:
                fprintf(stdout, "' %lf;\n", json_object_get_double(obj));
                break;
+       case json_type_null:
+               fprintf(stdout, "';\n");
+               break;
        default:
                return -1;
        }
@@ -240,6 +243,8 @@ static void jshn_add_object_var(json_object *obj, bool array, const char *prefix
                new = json_object_new_double(strtod(var, NULL));
        } else if (!strcmp(type, "boolean")) {
                new = json_object_new_boolean(!!atoi(var));
+       } else if (!strcmp(type, "null")) {
+               new = NULL;
        } else {
                return;
        }
index bf76edb..1090814 100644 (file)
@@ -168,6 +168,12 @@ json_add_double() {
        _json_add_generic double "$1" "$2" "$cur"
 }
 
+json_add_null() {
+       local cur
+       _json_get_var cur JSON_CUR
+       _json_add_generic null "$1" "" "$cur"
+}
+
 # functions read access to json variables
 
 json_load() {