On 9/16/20 8:25 PM, Eduardo Habkost wrote: > The requirement to specify the parent class type makes the macro > harder to use and easy to misuse (silent bugs can be introduced > if the wrong struct type is specified). > > Simplify the macro by just not declaring any class struct, > allowing us to remove the class_size field from the TypeInfo > variables for those types. > > Signed-off-by: Eduardo Habkost <[email protected]> > --- > Cc: "Daniel P. Berrangé" <[email protected]> > Cc: "Marc-André Lureau" <[email protected]> > Cc: "Michael S. Tsirkin" <[email protected]> > Cc: Paolo Bonzini <[email protected]> > Cc: Eduardo Habkost <[email protected]> > Cc: Gerd Hoffmann <[email protected]> > Cc: [email protected] > --- > include/authz/list.h | 2 +- > include/authz/listfile.h | 2 +- > include/authz/pamacct.h | 2 +- > include/authz/simple.h | 2 +- > include/crypto/secret_keyring.h | 2 +- > include/io/dns-resolver.h | 2 +- > include/io/net-listener.h | 2 +- > include/qom/object.h | 16 ++++++++-------- > include/sysemu/vhost-user-backend.h | 2 +- > authz/list.c | 1 - > authz/listfile.c | 1 - > authz/pamacct.c | 1 - > authz/simple.c | 1 - > backends/dbus-vmstate.c | 3 +-- > backends/vhost-user.c | 1 - > crypto/secret_keyring.c | 1 - > io/dns-resolver.c | 1 - > io/net-listener.c | 1 - > ui/input-barrier.c | 3 +-- > ui/input-linux.c | 3 +-- > 20 files changed, 19 insertions(+), 30 deletions(-)
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
