[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3] qobject: json-streamer: Change bracket_count to brace_cou
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v3] qobject: json-streamer: Change bracket_count to brace_count |
Date: |
Tue, 07 Apr 2020 13:42:05 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Markus Armbruster <address@hidden> writes:
> Simran Singhal <address@hidden> writes:
>
>> Change the "parser->bracket_count >= 0" to "parser->brace_count >= 0"
>> to fix the typo.
>>
>> Fixes: 8d3265b3
>>
>> Reviewed-by: Eric Blake <address@hidden>
>> Signed-off-by: Simran Singhal <address@hidden>
>
> First, thank you so much for spotting and fixing my typo!
>
> The commit message neglects to explain what is broken. I'd like to use
> this one:
>
> json: Fix check for unbalanced right curly brace
>
> We immediately diagnose unbalanced right curly brace:
>
> $ qemu-kvm --nodefaults --nographic --qmp stdio
> {"QMP": {"version": {"qemu": {"micro": 91, "minor": 2, "major": 4},
> "package": "v5.0.0-rc1-1-gf6ce4a439a08"}, "capabilities": ["oob"]}}
> }
> {"error": {"class": "GenericError", "desc": "JSON parse error,
> expecting value"}}
>
> except within square bracket:
>
> [}
>
> The check for unbalanced braces has a typo. Fix it.
>
> Fixes: 8d3265b3d00db1071d1d3bf8433b4818088fdeb5
> Signed-off-by: Simran Singhal <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
>
> Objections?
Apparently none. Queued, thanks!
[...]
- [PATCH v3] qobject: json-streamer: Change bracket_count to brace_count, Simran Singhal, 2020/04/02
- Re: [PATCH v3] qobject: json-streamer: Change bracket_count to brace_count, no-reply, 2020/04/02
- Re: [PATCH v3] qobject: json-streamer: Change bracket_count to brace_count, no-reply, 2020/04/02
- Re: [PATCH v3] qobject: json-streamer: Change bracket_count to brace_count, no-reply, 2020/04/02
- Re: [PATCH v3] qobject: json-streamer: Change bracket_count to brace_count, no-reply, 2020/04/02
- Re: [PATCH v3] qobject: json-streamer: Change bracket_count to brace_count, no-reply, 2020/04/02
- Re: [PATCH v3] qobject: json-streamer: Change bracket_count to brace_count, Markus Armbruster, 2020/04/03
- Re: [PATCH v3] qobject: json-streamer: Change bracket_count to brace_count,
Markus Armbruster <=