The event test does not try to include libvirt internals. Using a macro
named VIR_DEBUG might hint to such usage. Additionally it's useless
since it's used only in the main() function.

Modernize the message strings while touching them.
---
 examples/object-events/event-test.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/examples/object-events/event-test.c 
b/examples/object-events/event-test.c
index cde2d83..e3f0330 100644
--- a/examples/object-events/event-test.c
+++ b/examples/object-events/event-test.c
@@ -9,7 +9,6 @@
 #include <libvirt/libvirt.h>
 #include <libvirt/virterror.h>

-#define VIR_DEBUG(fmt) printf("%s:%d: " fmt "\n", __func__, __LINE__)
 #define STREQ(a, b) (strcmp(a, b) == 0)

 #ifndef ATTRIBUTE_UNUSED
@@ -656,7 +655,7 @@ int main(int argc, char **argv)
     sigaction(SIGTERM, &action_stop, NULL);
     sigaction(SIGINT, &action_stop, NULL);

-    VIR_DEBUG("Registering event cbs");
+    printf("Registering event callbacks\n");

     /* Add 2 callbacks to prove this works with more than just one */
     callback1ret = virConnectDomainEventRegister(dconn, myDomainEventCallback1,
@@ -787,7 +786,7 @@ int main(int argc, char **argv)
         }
     }

-    VIR_DEBUG("Deregistering event handlers");
+    printf("Deregistering event callbacks\n");
     virConnectDomainEventDeregister(dconn, myDomainEventCallback1);
     virConnectDomainEventDeregisterAny(dconn, callback2ret);
     virConnectDomainEventDeregisterAny(dconn, callback3ret);
@@ -816,9 +815,9 @@ int main(int argc, char **argv)

  cleanup:
     if (dconn) {
-        VIR_DEBUG("Closing connection");
+        printf("Closing connection: ");
         if (virConnectClose(dconn) < 0)
-            printf("error closing\n");
+            printf("failed\n");
         printf("done\n");
     }

-- 
2.8.0

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to