[PATCH v2 1/9] pulse: Add a JSON-parsing library

[arun@xxxxxxxxxxxxxxxx (Arun Raghavan)]

On Thu, 2 Jun 2016, at 04:41 PM, Tanu Kaskinen wrote:
> On Wed, 2016-06-01 at 17:18 +0530, Arun Raghavan wrote:
> > +static const char* parse_string(const char *str, pa_json_object *obj) {
> > +Â Â Â Â pa_strbuf *buf = pa_strbuf_new();
> > +
> > +Â Â Â Â str++; /* Consume leading '"' */
> > +
> > +Â Â Â Â while (*str != '"') {
> > +Â Â Â Â Â Â Â Â if (*str != '\\') {
> > +Â Â Â Â Â Â Â Â Â Â Â Â /* We don't accept non-ASCII, non-control characters */
> 
> This comment seems to be saying either that we don't accept any
> printable characters, or that we don't accept non-ASCII printable
> characters. Rewording suggestion: "We only accept ASCII printable
> characters."
> 
> > +Â Â Â Â Â Â Â Â Â Â Â Â if (*str < 0x20) {
> 
> Should be "if (*str < 0x20 || *str > 0x7E) {"

Actually, no. That should throw a compiler warning since str is a
(signed) char and will never be greater than 0x7E. Maybe we can just
cast to unsigned and do the check so this isn't confusing.

-- Arun