On 6/19/2019 5:58 AM, Nguyễn Thái Ngọc Duy wrote:
> + if (jw) {
> + jw_object_inline_begin_object(jw, "resolve-undo");
> + jw_object_intmax(jw, "ext-size", size);
> + jw_object_inline_begin_array(jw, "entries");
> + }
While reading this block, I noticed the use of hyphens in the
member names could cause some problems when translating into
object models in some languages. While this is valid JSON, I
found helpful recommendations in the Google JSON Style Guide [1]
that could apply here and elsewhere. Specifically, this
recommendation:
"Property names must be camel-cased, ascii strings."
Treating JSON members as camel-cased variable names would
promote consumption by third-party tools.
Thanks,
-Stolee
[1]
https://google.github.io/styleguide/jsoncstyleguide.xml?showone=Property_Name_Format#Property_Name_Format
