Hi On Mon, Dec 21, 2015 at 6:08 PM, Eric Blake <ebl...@redhat.com> wrote: > The generator special-cased > { 'command':'foo', 'data': {} } > to avoid emitting a visitor variable, but failed to see that > { 'struct':'NamedEmptyType, 'data': {} } > { 'command':'foo', 'data':'NamedEmptyType' } > needs the same treatment. Without a fix to the generator, the > change to qapi-schema-test.json fails to compile with: > > tests/test-qmp-marshal.c: In function ‘qmp_marshal_user_def_cmd0’: > tests/test-qmp-marshal.c:264:14: error: variable ‘v’ set but not used > [-Werror=unused-but-set-variable] > Visitor *v; > ^ > > Signed-off-by: Eric Blake <ebl...@redhat.com>
Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com> -- Marc-André Lureau