On 06/17/2016 01:54 AM, Markus Armbruster wrote:
>>>
>>> I'll take this one through qapi-next. Thanks!
>>>
>>
>> You may want to squash this in for shorter lines:
>>
>>
>> diff --git a/qobject/json-lexer.c b/qobject/json-lexer.c
>> index de16219..b030576 100644
>> --- a/qobject/json-lexer.c
>> ++
Eric Blake writes:
> On 06/16/2016 10:19 AM, Markus Armbruster wrote:
>> Eric Blake writes:
>>
>>> Fix the regex comments describing what we parse as JSON. No change
>>> to the lexer itself, just to the comments:
>>> - The "" and '' string construction was missing alternation between
>>> diffe
On 06/16/2016 10:19 AM, Markus Armbruster wrote:
> Eric Blake writes:
>
>> Fix the regex comments describing what we parse as JSON. No change
>> to the lexer itself, just to the comments:
>> - The "" and '' string construction was missing alternation between
>> different escape sequences
>> -
Eric Blake writes:
> Fix the regex comments describing what we parse as JSON. No change
> to the lexer itself, just to the comments:
> - The "" and '' string construction was missing alternation between
> different escape sequences
> - The construction for numbers forgot to handle optional leadi
Fix the regex comments describing what we parse as JSON. No change
to the lexer itself, just to the comments:
- The "" and '' string construction was missing alternation between
different escape sequences
- The construction for numbers forgot to handle optional leading '-'
- The construction for n