[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 0/6]: QMP: Fix issues in parser/lexer
From: |
Luiz Capitulino |
Subject: |
[Qemu-devel] Re: [PATCH 0/6]: QMP: Fix issues in parser/lexer |
Date: |
Thu, 20 May 2010 12:26:19 -0300 |
On Thu, 20 May 2010 17:18:23 +0200
Paolo Bonzini <address@hidden> wrote:
> On 05/19/2010 11:43 PM, Anthony Liguori wrote:
> >
> >> 4. Lexer expects a 'terminal' char to process a token
> >>
> >> Which means clients must send a sort of end of line char, so that we
> >> process their input.
> >>
> >> Maybe I'm missing something here, but I thought that the whole
> >> point of writing our own parser was to avoid this.
> >
> > If the lexer gets:
> >
> > "abc"
> >
> > It has no way of knowing if that's a token or if we're going to get:
> >
> > "abcd"
>
> Only } and ] are valid characters at the end of a JSON object, and
> neither requires lookahead.
Good point.
- Re: [Qemu-devel] Re: [PATCH 2/6] json-lexer: Handle missing escapes, (continued)
[Qemu-devel] [PATCH 3/6] qjson: Handle "\f", Luiz Capitulino, 2010/05/19
[Qemu-devel] [PATCH 4/6] check-qjson: Add more escape tests, Luiz Capitulino, 2010/05/19
[Qemu-devel] [PATCH 6/6] json-streamer: Don't use qdict_put_obj(), Luiz Capitulino, 2010/05/19
[Qemu-devel] [PATCH 5/6] json-lexer: Drop 'buf', Luiz Capitulino, 2010/05/19
Re: [Qemu-devel] [PATCH 0/6]: QMP: Fix issues in parser/lexer, Anthony Liguori, 2010/05/19
- Re: [Qemu-devel] [PATCH 0/6]: QMP: Fix issues in parser/lexer, Luiz Capitulino, 2010/05/20
- [Qemu-devel] Re: [PATCH 0/6]: QMP: Fix issues in parser/lexer, Paolo Bonzini, 2010/05/20
- [Qemu-devel] Re: [PATCH 0/6]: QMP: Fix issues in parser/lexer,
Luiz Capitulino <=
- [Qemu-devel] Re: [PATCH 0/6]: QMP: Fix issues in parser/lexer, Anthony Liguori, 2010/05/20
- [Qemu-devel] Re: [PATCH 0/6]: QMP: Fix issues in parser/lexer, Luiz Capitulino, 2010/05/20
- [Qemu-devel] [PATCH] do not require lookahead in json-lexer.c if not necessary, Paolo Bonzini, 2010/05/21
- [Qemu-devel] [PATCH] do not require lookahead for escapes too, Paolo Bonzini, 2010/05/21
- [Qemu-devel] Re: [PATCH] do not require lookahead for escapes too, Paolo Bonzini, 2010/05/23
Re: [Qemu-devel] [PATCH 0/6]: QMP: Fix issues in parser/lexer, Avi Kivity, 2010/05/20