Fix issue where JSON.stringify({ something: bool }) would output 'something' as a bool too

Author: gfwilliams

ChangeLog

@@ -13,6 +13,7 @@
             JSON.stringify now checks for potential stack overflow when stringifying (fix #1940)
             Check for Stack overflow when Garbage Collecting giant linked list (fix #1765)
             Added String.padStart/padEnd
+            Fix issue where JSON.stringify({ something: bool }) would output 'something' as a bool too
 
      2v07 : Graphics.asImage() now contains buffer when bpp != 8 (fix #1863)
             nRF52 SDK15: Fix NRF.setScan/findDevices/etc

src/jsvar.c

@@ -99,7 +99,7 @@ bool jsvIsName(const JsVar *v) { return v && (v->flags&JSV_VARTYPEMASK)>=_JSV_NA
 bool jsvIsBasicName(const JsVar *v) { return v && (v->flags&JSV_VARTYPEMASK)>=JSV_NAME_STRING_0 && (v->flags&JSV_VARTYPEMASK)<=JSV_NAME_STRING_MAX; } ///< Simple NAME that links to a variable via firstChild
 /// Names with values have firstChild set to a value - AND NOT A REFERENCE
 bool jsvIsNameWithValue(const JsVar *v) { return v && (v->flags&JSV_VARTYPEMASK)>=_JSV_NAME_WITH_VALUE_START && (v->flags&JSV_VARTYPEMASK)<=_JSV_NAME_WITH_VALUE_END; }
-bool jsvIsNameInt(const JsVar *v) { return v && ((v->flags&JSV_VARTYPEMASK)==JSV_NAME_INT_INT || ((v->flags&JSV_VARTYPEMASK)>=JSV_NAME_STRING_INT_0 && (v->flags&JSV_VARTYPEMASK)<=JSV_NAME_STRING_INT_MAX)); }
+bool jsvIsNameInt(const JsVar *v) { return v && ((v->flags&JSV_VARTYPEMASK)==JSV_NAME_INT_INT || ((v->flags&JSV_VARTYPEMASK)>=JSV_NAME_STRING_INT_0 && (v->flags&JSV_VARTYPEMASK)<=JSV_NAME_STRING_INT_MAX)); } ///< Is this a NAME pointing to an Integer value
 bool jsvIsNameIntInt(const JsVar *v) { return v && (v->flags&JSV_VARTYPEMASK)==JSV_NAME_INT_INT; }
 bool jsvIsNameIntBool(const JsVar *v) { return v && (v->flags&JSV_VARTYPEMASK)==JSV_NAME_INT_BOOL; }
 /// What happens when we access a variable that doesn't exist. We get a NAME where the next + previous siblings point to the object that may one day contain them
@@ -1232,7 +1232,7 @@ const char *jsvGetConstString(const JsVar *v) {
     return "undefined";
   } else if (jsvIsNull(v)) {
     return "null";
-  } else if (jsvIsBoolean(v)) {
+  } else if (jsvIsBoolean(v) && !jsvIsNameIntBool(v)) {
     return jsvGetBool(v) ? "true" : "false";
   }
   return 0;

src/jsvar.h

@@ -429,7 +429,7 @@ extern bool jsvIsName(const JsVar *v); ///< NAMEs are what's used to name a vari
 bool jsvIsBasicName(const JsVar *v); ///< Simple NAME that links to a variable via firstChild
 /// Names with values have firstChild set to a value - AND NOT A REFERENCE
 extern bool jsvIsNameWithValue(const JsVar *v);
-extern bool jsvIsNameInt(const JsVar *v);
+extern bool jsvIsNameInt(const JsVar *v); ///< Is this a NAME pointing to an Integer value
 extern bool jsvIsNameIntInt(const JsVar *v);
 extern bool jsvIsNameIntBool(const JsVar *v);
 /// What happens when we access a variable that doesn't exist. We get a NAME where the next + previous siblings point to the object that may one day contain them