pkarashchenko commented on code in PR #1590: URL: https://github.com/apache/nuttx-apps/pull/1590#discussion_r1116065032
########## interpreters/bas/bas_value.c: ########## @@ -2026,9 +2042,21 @@ struct String *Value_toWrite(struct Value *this, struct String *s) struct Value *Value_nullValue(enum ValueType type) { - static struct Value integer = { V_INTEGER }; - static struct Value real = { V_REAL }; - static struct Value string = { V_STRING }; + static struct Value integer = + { + V_INTEGER + }; + + static struct Value real = + { + V_REAL + }; + + static struct Value string = + { + V_STRING + }; + static char n[] = ""; Review Comment: Can we remove `n` and use `""` directly? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org