Re: [Qemu-devel] [PATCH 02/26] qapi: Clean up cgen() and mcgen()

2015-08-04 Thread Eric Blake
On 08/04/2015 09:44 AM, Markus Armbruster wrote: > Markus Armbruster writes: > >> Commit 05dfb26 added eatspace stripping to mcgen(). Move it to >> cgen(), just in case somebody gets tempted to use cgen() directly >> instead of via mcgen(). >> >> Output doesn't change >> >> Signed-off-by: Marku

Re: [Qemu-devel] [PATCH 02/26] qapi: Clean up cgen() and mcgen()

2015-08-04 Thread Markus Armbruster
Markus Armbruster writes: > Commit 05dfb26 added eatspace stripping to mcgen(). Move it to > cgen(), just in case somebody gets tempted to use cgen() directly > instead of via mcgen(). > > cgen() indents blank lines. No such lines get generated right now, > but fix it anyway. > > We use triple-

[Qemu-devel] [PATCH 02/26] qapi: Clean up cgen() and mcgen()

2015-08-04 Thread Markus Armbruster
Commit 05dfb26 added eatspace stripping to mcgen(). Move it to cgen(), just in case somebody gets tempted to use cgen() directly instead of via mcgen(). cgen() indents blank lines. No such lines get generated right now, but fix it anyway. We use triple-quoted strings for program text, like this