[Libvirt-ci] Broken: libvirt/libvirt#936 (master - 67966ad)
Travis CI
builds at travis-ci.org
Wed Feb 21 17:41:52 UTC 2018
Build Update for libvirt/libvirt
-------------------------------------
Build: #936
Status: Broken
Duration: 25 minutes and 10 seconds
Commit: 67966ad (master)
Author: Daniel P. Berrangé
Message: m4: enforce that all enum cases are listed in switch statements
As a general rule any time we switch() on something that is an enum, we
want to have a case for every enum constant. The -Wswitch warning will
report any switch where we've violated this rule, except if that switch
has a default case.
Unfortunately it is reasonable to want to list all enum constants *and*
also have a default case. To get a warning in that scenario requires
that we turn on -Wswitch-enum.
In a few cases where we explicitly don't want to list all enum cases, we
can discard the enum type checking by casting the value to a plain int.
Reviewed-by: John Ferlan <jferlan at redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
View the changeset: https://github.com/libvirt/libvirt/compare/f565321b26df...67966ad51819
View the full build log and details: https://travis-ci.org/libvirt/libvirt/builds/344421390?utm_source=email&utm_medium=notification
--
You can configure recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvirt-ci/attachments/20180221/8adf395f/attachment.htm>
More information about the Libvirt-ci
mailing list