Re: [PATCH 4/8] resolve-undo.c: dump "REUC" extension as json

From: Derrick Stolee <stolee@gmail.com>
Date: Jun 19, 2019, 13:16 UTC
Message-ID: <3dba92fa-68ce-8204-fdfe-a0435528a9c2@gmail.com>
In-Reply-To: <20190619095858.30124-5-pclouds@gmail.com>

On 6/19/2019 5:58 AM, Nguyễn Thái Ngọc Duy wrote:

Show 5 quoted lines

> +	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