]> xenbits.xensource.com Git - libvirt.git/commitdiff
event-test: Get rid of useless and ambiguous VIR_DEBUG macro
authorPeter Krempa <pkrempa@redhat.com>
Wed, 13 Apr 2016 11:55:12 +0000 (13:55 +0200)
committerPeter Krempa <pkrempa@redhat.com>
Mon, 18 Apr 2016 14:31:27 +0000 (16:31 +0200)
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

index cde2d83b3d94fed5cc892f06a5a69b9c9654a2b3..e3f03309058bfcd8f5a3d8b5961a75546bdd3b03 100644 (file)
@@ -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");
     }