@yuneta/gobj-js 7.25.2 → 7.25.4

package/README.md

@@ -531,8 +531,17 @@ kw_get_dict_value(gobj, kw, path, default_value, flag)
 // the default AS GIVEN otherwise (absent key or other type), as in C:
 //   kw_get_list(gobj, {x: [1, 2]}, "x", [], 0)   → [1, 2]  (the same array)
 //   kw_get_dict(gobj, {}, "x", null, 0)          → null
-//   kw_get_bool(gobj, {x: "false"}, "x", true, 0) → true   (not a boolean)
+//   kw_get_bool(gobj, {x: "false"}, "x", true, 0) → true   (not a boolean: logged)
 //   kw_get_bool(gobj, {x: "false"}, "x", true, KW_WILD_NUMBER) → false
+// kw_get_bool() logs a value that is not a boolean even without
+// KW_REQUIRED ("path MUST BE a json boolean"), as the C reader does,
+// and so do kw_get_int() / kw_get_real() for a value that is not a
+// number ("path MUST BE a json integer" / "... a json real"):
+//   kw_get_int(gobj, {x: 3.9}, "x", 0, 0)               → 3  (truncated, as C)
+//   kw_get_int(gobj, {x: "12"}, "x", 5, 0)              → 5  (not a number: logged)
+//   kw_get_int(gobj, {x: "0x1F"}, "x", 5, KW_WILD_NUMBER) → 31 (strtoll base 0)
+// KW_EXTRACT takes out only a value the reader answers with: a value
+// of another type stays in the kw.
 
 kw_set_dict_value(gobj, kw, path, value)
 kw_set_subdict_value(gobj, kw, path, key, value)

package/dist/gobj-js.cjs.js

@@ -3605,10 +3605,13 @@ function kw_delete(gobj, kw, path) {
 }
 /************************************************************
 *  As the C kw_get_bool(): a boolean found is the answer, and
-*  anything else gives the default back -- unless KW_WILD_NUMBER
-*  asks to read a number (0 is false), a string ("true"/"false"
-*  in any case, else its integer) or null (false). Before gobj-js
-*  7.25.2 the value went through Boolean(), so "false" was TRUE.
+*  anything else gives the default back, LOGGED ("path MUST BE a
+*  json boolean", required or not) -- unless KW_WILD_NUMBER asks
+*  to read a number (0 is false), a string ("true"/"false" in any
+*  case, else its integer) or null (false); a list or a dict is
+*  then false, logged too. Before gobj-js 7.25.2 the value went
+*  through Boolean(), so "false" was TRUE; before 7.25.3 a value
+*  of the wrong type gave the default back in silence.
 ************************************************************/
 function kw_get_bool(gobj, kw, path, default_value, flag) {
 	if (gobj && !is_gobj(gobj)) log_error(`GObj bad instanceof`);
@@ -3636,18 +3639,47 @@ function kw_get_bool(gobj, kw, path, default_value, flag) {
 		else if (b.toLowerCase() === "false") value = false;
 		else value = (parseInt(b) || 0) !== 0;
 	} else if (wild && b === null) value = false;
-	else {
-		if (required) {
-			log_error(`${gobj_short_name(gobj)}: path value MUST BE a boolean: ${path}`);
-			trace_msg(kw);
-		}
+	else if (wild) {
+		log_error(`${gobj ? gobj_short_name(gobj) + ": " : ""}path MUST BE a simple json element: '${path}'`);
+		trace_json(kw);
+		return false;
+	} else {
+		log_error(`${gobj ? gobj_short_name(gobj) + ": " : ""}path MUST BE a json boolean: '${path}'`);
+		trace_json(kw);
 		return Boolean(default_value);
 	}
 	if (extract) kw_delete(gobj, kw, path);
 	return value;
 }
 /************************************************************
-*
+*  The integer at the start of a string, as C's strtoll(s, 0, 0)
+*  reads it: leading spaces, a sign, then "0x" for hex, "0" for
+*  octal, else decimal, up to the first character that is not a
+*  digit of that base. 0 when there is no digit.
+************************************************************/
+function strtoll_base0(s) {
+	const m = /^\s*([+-]?)(0[xX][0-9a-fA-F]+|0[0-7]*|[1-9][0-9]*)/.exec(s);
+	if (!m) return 0;
+	let digits = m[2];
+	let value;
+	if (digits.length > 1 && (digits[1] === "x" || digits[1] === "X")) value = parseInt(digits.substring(2), 16);
+	else if (digits[0] === "0") value = parseInt(digits, 8);
+	else value = parseInt(digits, 10);
+	return m[1] === "-" ? -value : value;
+}
+/************************************************************
+*  As the C kw_get_int(): a number found is the answer,
+*  truncated toward zero as C casts a real, and anything else
+*  gives the default back (as an integer), LOGGED ("path MUST BE
+*  a json integer", required or not) -- unless KW_WILD_NUMBER
+*  asks to read a boolean (1/0), a string (as strtoll() with base
+*  0: "0x1F" is 31, "017" is 15, "12abc" is 12, "abc" is 0) or
+*  null (0); a list or a dict is then 0, logged too. KW_EXTRACT
+*  takes out only a value the reader answers with.
+*  Before gobj-js 7.25.4 KW_EXTRACT deleted the value before its
+*  type was looked at, a value of another type was logged only
+*  with KW_REQUIRED, KW_WILD_NUMBER was ignored, and a number went
+*  through parseInt(), which reads it as a string: 1e-7 gave 1.
 ************************************************************/
 function kw_get_int(gobj, kw, path, default_value, flag) {
 	if (gobj && !is_gobj(gobj)) log_error(`GObj bad instanceof`);
@@ -3655,6 +3687,7 @@ function kw_get_int(gobj, kw, path, default_value, flag) {
 	const required = flag && flag & kw_flag_t.KW_REQUIRED;
 	const create = flag && flag & kw_flag_t.KW_CREATE;
 	const extract = flag && flag & kw_flag_t.KW_EXTRACT;
+	const wild = flag && flag & kw_flag_t.KW_WILD_NUMBER;
 	if (v === void 0) {
 		if (create) {
 			let v = parseInt(default_value);
@@ -3666,18 +3699,34 @@ function kw_get_int(gobj, kw, path, default_value, flag) {
 		}
 		return parseInt(default_value);
 	}
-	if (extract) kw_delete(gobj, kw, path);
-	if (!is_number(v)) {
-		if (required) {
-			log_error(`path value MUST BE a number: ${path}`);
-			trace_msg(kw);
-		}
+	let value;
+	if (is_number(v)) value = Math.trunc(v);
+	else if (!wild) {
+		log_error(`${gobj ? gobj_short_name(gobj) + ": " : ""}path MUST BE a json integer: '${path}'`);
+		trace_json(kw);
 		return parseInt(default_value);
+	} else if (is_boolean(v)) value = v ? 1 : 0;
+	else if (is_string(v)) value = strtoll_base0(v);
+	else if (v === null) value = 0;
+	else {
+		log_error(`${gobj ? gobj_short_name(gobj) + ": " : ""}path MUST BE a simple json element: '${path}'`);
+		trace_json(kw);
+		return 0;
 	}
-	return parseInt(v);
+	if (extract) kw_delete(gobj, kw, path);
+	return value;
 }
 /************************************************************
-*
+*  As the C kw_get_real(): a number found is the answer, and
+*  anything else gives the default back (as a number), LOGGED
+*  ("path MUST BE a json real", required or not) -- unless
+*  KW_WILD_NUMBER asks to read a boolean (1/0), a string (as
+*  parseFloat() reads it, 0 when it does not start with a number)
+*  or null (0); a list or a dict is then 0, logged too.
+*  KW_EXTRACT takes out only a value the reader answers with.
+*  Before gobj-js 7.25.4 KW_EXTRACT deleted the value before its
+*  type was looked at, a value of another type was logged only
+*  with KW_REQUIRED, and KW_WILD_NUMBER was ignored.
 ************************************************************/
 function kw_get_real(gobj, kw, path, default_value, flag) {
 	if (gobj && !is_gobj(gobj)) log_error(`GObj bad instanceof`);
@@ -3685,6 +3734,7 @@ function kw_get_real(gobj, kw, path, default_value, flag) {
 	const required = flag && flag & kw_flag_t.KW_REQUIRED;
 	const create = flag && flag & kw_flag_t.KW_CREATE;
 	const extract = flag && flag & kw_flag_t.KW_EXTRACT;
+	const wild = flag && flag & kw_flag_t.KW_WILD_NUMBER;
 	if (v === void 0) {
 		if (create) {
 			let v = Number(default_value);
@@ -3696,15 +3746,24 @@ function kw_get_real(gobj, kw, path, default_value, flag) {
 		}
 		return Number(default_value);
 	}
-	if (extract) kw_delete(gobj, kw, path);
-	if (!is_number(v)) {
-		if (required) {
-			log_error(`path value MUST BE a number: ${path}`);
-			trace_msg(kw);
-		}
+	let value;
+	if (is_number(v)) value = v;
+	else if (!wild) {
+		log_error(`${gobj ? gobj_short_name(gobj) + ": " : ""}path MUST BE a json real: '${path}'`);
+		trace_json(kw);
 		return Number(default_value);
+	} else if (is_boolean(v)) value = v ? 1 : 0;
+	else if (is_string(v)) {
+		value = parseFloat(v);
+		if (Number.isNaN(value)) value = 0;
+	} else if (v === null) value = 0;
+	else {
+		log_error(`${gobj ? gobj_short_name(gobj) + ": " : ""}path MUST BE a simple json element: '${path}'`);
+		trace_json(kw);
+		return 0;
 	}
-	return Number(v);
+	if (extract) kw_delete(gobj, kw, path);
+	return value;
 }
 /************************************************************
 *  The default comes back AS GIVEN, as in the C kw_get_str()
@@ -3713,6 +3772,11 @@ function kw_get_real(gobj, kw, path, default_value, flag) {
 *  TRUE: an `if(value)` on the answer took an absent key for a
 *  present one. KW_CREATE stores a string default, and null for
 *  any other, as C stores json_null() for a NULL default.
+*  KW_EXTRACT takes out only a string (before gobj-js 7.25.4 it
+*  deleted any value, and a value that was not a string was lost).
+*  C refuses to extract a string at all ("Cannot extract a
+*  string": its answer would point into the freed json); a JS
+*  string outlives the kw, so here it is taken out.
 ************************************************************/
 function kw_get_str(gobj, kw, path, default_value, flag) {
 	if (gobj && !is_gobj(gobj)) log_error(`GObj bad instanceof`);
@@ -3730,7 +3794,6 @@ function kw_get_str(gobj, kw, path, default_value, flag) {
 		}
 		return default_value;
 	}
-	if (extract) kw_delete(gobj, kw, path);
 	if (!is_string(v)) {
 		if (required) {
 			log_error(`${gobj_short_name(gobj)}: path value MUST BE a string: ${path}`);
@@ -3738,6 +3801,7 @@ function kw_get_str(gobj, kw, path, default_value, flag) {
 		}
 		return default_value;
 	}
+	if (extract) kw_delete(gobj, kw, path);
 	return v;
 }
 /************************************************************