[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH 3/6] conf: Make switch statements more strict

On 02/21/2017 02:57 PM, Andrea Bolognani wrote:
When switching over the values in the virDomainControllerModelPCI
enumeration, make sure the proper cast is in place so that the
compiler can warn us when the coverage is not exaustive.

For the same reason, remove the 'default' case from one of the
existing switch statements.
  src/conf/domain_addr.c |  4 ++--
  src/conf/domain_conf.c | 12 +++++++++++-
  2 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/src/conf/domain_addr.c b/src/conf/domain_addr.c
index 519cc6b..933c2fe 100644
--- a/src/conf/domain_addr.c
+++ b/src/conf/domain_addr.c
@@ -344,9 +344,9 @@ virDomainPCIAddressBusSetModel(virDomainPCIAddressBusPtr bus,
          bus->maxSlot = VIR_PCI_ADDRESS_SLOT_LAST;
- default:
-                       _("Invalid PCI controller model %d"), model);
+                       _("PCI controller model was not set correctly"));

You're going to need to precede the string with "%s", to avoid a syntax-check failure.

ACK with that fixed.

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]