[libvirt] [PATCH] Don't use enums in TPM struct fields
Daniel P. Berrangé
berrange at redhat.com
Thu Jun 7 08:33:25 UTC 2018
On Thu, Jun 07, 2018 at 10:25:11AM +0200, Peter Krempa wrote:
> On Thu, Jun 07, 2018 at 09:15:06 +0100, Daniel Berrange wrote:
> > On Wed, Jun 06, 2018 at 08:32:39PM +0200, Ján Tomko wrote:
> > > This deprives us of the -Wswitch-enum warning on all compilers
> > > because some don't detect the bogus negative value comparison.
> > > And the comment has even less power than the clang warning. So:
> > >
> > > 1. Is it actually worth the trouble to store enum values in
> > > typedef'd enums?
> > > 2. If so, can we make TypeFromString usage less cumbersome?
> >
> > We could add a explicit
> >
> > VIR_XXXXX_INVALID = -1,
> >
> > entry to every single enum, which will force the compiler to
> > always use a signed int for representing the enum.
>
> That would force us to add that to all switch statements with the
> correct type, which seems a waste.
Yeah, that would be annoying, though no different to the need to add
_LAST to every switch too.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
More information about the libvir-list
mailing list