On 03/27/2015 03:52 AM, Markus Armbruster wrote: > Eric Blake writes: > >> In the testsuite, UserDefTwo and UserDefNested were identical >> types other than the member names. Reduce code duplication by >> having just one type, and choose names that also favor reuse. >> This will also make it easier for a later patch to get rid of >> inline nested types in QAPI; it means that the type is now boxed >> instead of unboxed in C code, but has no difference to the QMP >> wire protocol. > > I can't see a change from boxed to unboxed in this patch. Does the > remark apply to the elimination of nested types? If yes, it's premature > here. May be a stale comment; either as the result of too much copy-and-paste between similar patches 25 and 26, or as the result of begin first written last September and several of your tree cleanups hitting the git repo in the meantime. I'll clean up the commit wording in v6. > >> When touching code to add new allocations, also >> convert existing allocations to consistently prefer typesafe >> g_new0 over g_malloc0. > > I can't see any change from g_malloc0() to g_new0(). Is this stale? > >> >> Ensure that 'make check-qapi-schema check-unit' still passes. >> >> Signed-off-by: Eric Blake > > -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org