jshn: do not export JSON_SEQ

[project/libubox.git] / sh / jshn.sh

diff --git a/sh/jshn.sh b/sh/jshn.sh
index 176063f..fbffe85 100644 (file)
--- a/sh/jshn.sh
+++ b/sh/jshn.sh
@@ -42,7 +42,7 @@ _json_inc() {
        # var=$1
        # dest=$2
 
-       eval "${JSON_PREFIX}$1=\$(( \${${JSON_PREFIX}$1:-0} + 1))${2:+; $2=\"\$${JSON_PREFIX}$1\"}"
+       let "${JSON_PREFIX}$1 += 1" "$2 = ${JSON_PREFIX}$1"
 }
 
 _json_add_generic() {
@@ -92,6 +92,7 @@ _json_close_table() {
 
        _json_get_var _s_cur JSON_CUR
        _json_get_var "${JSON_PREFIX}JSON_CUR" "UP_$_s_cur"
+       unset "${JSON_PREFIX}UP_$_s_cur"
 }
 
 json_set_namespace() {
@@ -103,7 +104,7 @@ json_set_namespace() {
 }
 
 json_cleanup() {
-       local unset
+       local unset tmp
 
        _json_get_var unset JSON_UNSET
        for tmp in $unset JSON_VAR; do
@@ -124,8 +125,8 @@ json_cleanup() {
 
 json_init() {
        json_cleanup
+       export -n ${JSON_PREFIX}JSON_SEQ=0
        export -- \
-               ${JSON_PREFIX}JSON_SEQ=0 \
                ${JSON_PREFIX}JSON_CUR="JSON_VAR" \
                ${JSON_PREFIX}KEYS_JSON_VAR= \
                ${JSON_PREFIX}TYPE_JSON_VAR=
@@ -174,7 +175,7 @@ json_add_double() {
 # functions read access to json variables
 
 json_load() {
-       eval `jshn -r "$1"`
+       eval "`jshn -r "$1"`"
 }
 
 json_dump() {
@@ -206,10 +207,11 @@ json_get_keys() {
 json_get_values() {
        local _v_dest="$1"
        local _v_keys _v_val _select=
+       local _json_no_warning=1
 
        unset "$_v_dest"
        [ -n "$2" ] && {
-               json_select "$2"
+               json_select "$2" || return 1
                _select=1
        }
 
@@ -231,13 +233,17 @@ json_get_var() {
 
        _json_get_var __cur JSON_CUR
        local __var="${JSON_PREFIX}${__cur}_${2//[^a-zA-Z0-9_]/_}"
-       eval "export -- \"$__dest=\${$__var}\"; [ -n \"\${$__var+x}\" ]"
+       eval "export -- \"$__dest=\${$__var:-$3}\"; [ -n \"\${$__var+x}\${3+x}\" ]"
 }
 
 json_get_vars() {
        while [ "$#" -gt 0 ]; do
                local _var="$1"; shift
-               json_get_var "$_var" "$_var"
+               if [ "$_var" != "${_var#*:}" ]; then
+                       json_get_var "${_var%%:*}" "${_var%%:*}" "${_var#*:}"
+               else
+                       json_get_var "$_var" "$_var"
+               fi
        done
 }
 
@@ -253,6 +259,7 @@ json_select() {
        [[ "$1" == ".." ]] && {
                _json_get_var cur JSON_CUR
                _json_get_var cur "UP_$cur"
+               unset "${JSON_PREFIX}UP_$cur"
                _json_set_var JSON_CUR "$cur"
                return 0
        }
@@ -260,10 +267,12 @@ json_select() {
        case "$type" in
                object|array)
                        json_get_var cur "$target"
+                       _json_get_var "${JSON_PREFIX}UP_$cur" JSON_CUR
                        _json_set_var JSON_CUR "$cur"
                ;;
                *)
-                       echo "WARNING: Variable '$target' does not exist or is not an array/object"
+                       [ -n "$_json_no_warning" ] || \
+                               echo "WARNING: Variable '$target' does not exist or is not an array/object"
                        return 1
                ;;
        esac