Eric Blake <ebl...@redhat.com> writes:

> qapi-commands has a nice helper gen_err_check(), but did not

but does not use

> use it everywhere. In fact, using it in more places makes it
> easier to reduce the lines of code used for generating error
> checks.  This in turn will make it easier for later patches
> to consolidate another common pattern among the generators.

Does this message need updating now you're using gen_err_check() less
aggressively?

> The generated code has one less blank line in qapi-event.c
> functions, but has no semantic difference.

Is it a stylistic improvement or the opposite?

> Signed-off-by: Eric Blake <ebl...@redhat.com>

Patch looks good.

Reply via email to