sh/jshn.sh: replace "tr" calls with inline substitution, signalize success with retur...
authorJo-Philipp Wich <jow@openwrt.org>
Tue, 29 May 2012 00:10:20 +0000 (02:10 +0200)
committerFelix Fietkau <nbd@openwrt.org>
Tue, 29 May 2012 00:13:15 +0000 (02:13 +0200)
sh/jshn.sh

index 7f922c4f41a3a62f0ebcef421b4bcf6e62a8c59c..a15cb00be9f7410e6a2984f6827e666fd1e48f50 100644 (file)
@@ -24,7 +24,7 @@ json_add_generic() {
                var=$(( ${aseq:-0} + 1 ))
                export -- "SEQ_$cur=$var"
        else
-               local name="$(echo -n "$var" | tr -C '[a-zA-Z0-9_]' _)"
+               local name="${var//[^a-zA-Z0-9_]/_}"
                [[ "$name" == "$var" ]] || export -- "NAME_${cur}_${name}=$var"
                var="$name"
        fi
@@ -94,14 +94,14 @@ json_dump() {
 
 json_get_type() {
        local dest="$1"
-       local var="$2"
-       eval "export -- \"$dest=\${TYPE_${JSON_CUR}_$var}\""
+       local var="TYPE_${JSON_CUR}_$2"
+       eval "[ -n \"\${$var+x}\" ] && export -- \"$dest=\${$var}\""
 }
 
 json_get_var() {
        local dest="$1"
-       local var="$(echo -n "$2" | tr -C '[a-zA-Z0-9_]' _)"
-       eval "export -- \"$dest=\${${JSON_CUR}_$var}\""
+       local var="${JSON_CUR}_${2//[^a-zA-Z0-9_]/_}"
+       eval "[ -n \"\${$var+x}\" ] && export -- \"$dest=\${$var}\""
 }
 
 json_get_vars() {
@@ -117,11 +117,11 @@ json_select() {
 
        [ -z "$1" ] && {
                JSON_CUR="JSON_VAR"
-               return
+               return 0
        }
        [[ "$1" == ".." ]] && {
                eval "JSON_CUR=\"\${UP_$JSON_CUR}\""
-               return;
+               return 0
        }
        json_get_type type "$target"
        case "$type" in
@@ -130,6 +130,7 @@ json_select() {
                ;;
                *)
                        echo "WARNING: Variable '$target' does not exist or is not an array/object"
+                       return 1
                ;;
        esac
 }