Make tonumber work on true and false jqlang#3112

davidfetter · May 7, 2024 · 4698694 · 4698694
1 parent ed8f715
commit 4698694
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 4 deletions.
diff --git a/src/builtin.c b/src/builtin.c
@@ -483,6 +483,12 @@ static jv f_tonumber(jq_state *jq, jv input) {
     jv_free(input);
     return number;
   }
+  if (jv_get_kind(input) == JV_KIND_FALSE) {
+	return jv_number(0);
+  }
+  if (jv_get_kind(input) == JV_KIND_TRUE) {
+	return jv_number(1);
+  }
   return type_error(input, "cannot be parsed as a number");
 }
 

diff --git a/tests/jq.test b/tests/jq.test
@@ -2038,8 +2038,8 @@ null
 2
 
 .[] |= try tonumber
-["1", "2a", "3", " 4 ", "5.67", ".89", "-876", "+5.43", 21]
-[1, 3, 5.67, 0.89, -876, 5.43, 21]
+["1", "2a", "3", " 4 ", "5.67", ".89", "-876", "+5.43", 21, true, false]
+[1, 3, 5.67, 0.89, -876, 5.43, 21, 1, 0]
 
 # Also 1859, but from 2073
 any(keys[]|tostring?;true)

diff --git a/tests/man.test b/tests/man.test