[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