Re: [Qemu-devel] [PATCH 4/9] qapi.py: Decent syntax error reporting

2013-07-26 Thread Eric Blake
On 07/26/2013 01:48 PM, Paolo Bonzini wrote: > Il 26/07/2013 21:33, Markus Armbruster ha scritto: >>> Do we even want to allow TABs in the schema files? Right now, they are >>> tab-free; if you error out here instead of futzing with tab width, we >>> could forcefully maintain that property. >> >>

Re: [Qemu-devel] [PATCH 4/9] qapi.py: Decent syntax error reporting

2013-07-26 Thread Paolo Bonzini
Il 26/07/2013 21:33, Markus Armbruster ha scritto: > > Do we even want to allow TABs in the schema files? Right now, they are > > tab-free; if you error out here instead of futzing with tab width, we > > could forcefully maintain that property. > > I'm not volunteering for the TAB police, but if

Re: [Qemu-devel] [PATCH 4/9] qapi.py: Decent syntax error reporting

2013-07-26 Thread Markus Armbruster
Eric Blake writes: > On 07/26/2013 06:39 AM, Markus Armbruster wrote: >> Signed-off-by: Markus Armbruster >> --- >> scripts/qapi.py | 29 +++-- >> tests/qapi-schema/test-qapi.py| 2 ++ >> tests/qapi-schema/unclosed-string.err | 2 +- >> 3

Re: [Qemu-devel] [PATCH 4/9] qapi.py: Decent syntax error reporting

2013-07-26 Thread Eric Blake
On 07/26/2013 06:39 AM, Markus Armbruster wrote: > Signed-off-by: Markus Armbruster > --- > scripts/qapi.py | 29 +++-- > tests/qapi-schema/test-qapi.py| 2 ++ > tests/qapi-schema/unclosed-string.err | 2 +- > 3 files changed, 30 insertions(

[Qemu-devel] [PATCH 4/9] qapi.py: Decent syntax error reporting

2013-07-26 Thread Markus Armbruster
Signed-off-by: Markus Armbruster --- scripts/qapi.py | 29 +++-- tests/qapi-schema/test-qapi.py| 2 ++ tests/qapi-schema/unclosed-string.err | 2 +- 3 files changed, 30 insertions(+), 3 deletions(-) diff --git a/scripts/qapi.py b/scripts/q