Re: [Qemu-devel] [PATCH] fixup! qapi: qapi.py: allow the ' character be escaped

2012-07-30 Thread Luiz Capitulino
On Sat, 28 Jul 2012 08:37:03 +0200 Markus Armbruster arm...@redhat.com wrote: Luiz Capitulino lcapitul...@redhat.com writes: On Thu, 26 Jul 2012 19:09:13 +0200 Markus Armbruster arm...@redhat.com wrote: Support escaping the escape character, and make more robust (don't die for '',

Re: [Qemu-devel] [PATCH] fixup! qapi: qapi.py: allow the ' character be escaped

2012-07-28 Thread Markus Armbruster
Luiz Capitulino lcapitul...@redhat.com writes: On Thu, 26 Jul 2012 19:09:13 +0200 Markus Armbruster arm...@redhat.com wrote: Support escaping the escape character, and make more robust (don't die for '', handle ' without matching '. Signed-off-by: Markus Armbruster arm...@redhat.com Can

Re: [Qemu-devel] [PATCH] fixup! qapi: qapi.py: allow the ' character be escaped

2012-07-27 Thread Luiz Capitulino
On Thu, 26 Jul 2012 19:09:13 +0200 Markus Armbruster arm...@redhat.com wrote: Support escaping the escape character, and make more robust (don't die for '', handle ' without matching '. Signed-off-by: Markus Armbruster arm...@redhat.com Can you please do you it on top of master so that I

[Qemu-devel] [PATCH] fixup! qapi: qapi.py: allow the ' character be escaped

2012-07-26 Thread Markus Armbruster
Support escaping the escape character, and make more robust (don't die for '', handle ' without matching '. Signed-off-by: Markus Armbruster arm...@redhat.com --- Luiz made me do this. Fixes up his 07/11. Generates identical qmp-commands.h qapi-types.h qapi-visit.h qapi-errors.h

Re: [Qemu-devel] [PATCH] fixup! qapi: qapi.py: allow the ' character be escaped

2012-07-26 Thread Peter Maydell
On 26 July 2012 18:09, Markus Armbruster arm...@redhat.com wrote: Support escaping the escape character, and make more robust (don't die for '', handle ' without matching '. Signed-off-by: Markus Armbruster arm...@redhat.com Yeah, let's do this the right way. Reviewed-by: Peter Maydell