From: Peter Krempa <pkrempa@redhat.com>
Date: Thu, 14 Apr 2016 12:44:07 +0000 (+0200)
Subject: event-test: Use switch instead of if/else if chains for lifecycle event translation
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=5ffc742c17faa569017a54cd2ddeef9e4c6d2cec;p=libvirt.git

event-test: Use switch instead of if/else if chains for lifecycle event translation
---

diff --git a/examples/object-events/event-test.c b/examples/object-events/event-test.c
index 9a53bb156b..b265e2e11f 100644
--- a/examples/object-events/event-test.c
+++ b/examples/object-events/event-test.c
@@ -91,20 +91,30 @@ eventDetailToString(int event,
     const char *ret = "";
     switch ((virDomainEventType) event) {
         case VIR_DOMAIN_EVENT_DEFINED:
-            if (detail == VIR_DOMAIN_EVENT_DEFINED_ADDED)
+            switch ((virDomainEventDefinedDetailType) detail) {
+            case VIR_DOMAIN_EVENT_DEFINED_ADDED:
                 ret = "Added";
-            else if (detail == VIR_DOMAIN_EVENT_DEFINED_UPDATED)
+                break;
+            case VIR_DOMAIN_EVENT_DEFINED_UPDATED:
                 ret = "Updated";
-            else if (detail == VIR_DOMAIN_EVENT_DEFINED_RENAMED)
+                break;
+            case VIR_DOMAIN_EVENT_DEFINED_RENAMED:
                 ret = "Renamed";
-            else if (detail == VIR_DOMAIN_EVENT_DEFINED_FROM_SNAPSHOT)
+                break;
+            case  VIR_DOMAIN_EVENT_DEFINED_FROM_SNAPSHOT:
                 ret = "Snapshot";
+                break;
+            }
             break;
         case VIR_DOMAIN_EVENT_UNDEFINED:
-            if (detail == VIR_DOMAIN_EVENT_UNDEFINED_REMOVED)
+            switch ((virDomainEventUndefinedDetailType) detail) {
+            case VIR_DOMAIN_EVENT_UNDEFINED_REMOVED:
                 ret = "Removed";
-            else if (detail == VIR_DOMAIN_EVENT_UNDEFINED_RENAMED)
+                break;
+            case VIR_DOMAIN_EVENT_UNDEFINED_RENAMED:
                 ret = "Renamed";
+                break;
+            }
             break;
         case VIR_DOMAIN_EVENT_STARTED:
             switch ((virDomainEventStartedDetailType) detail) {