From: Laine Stump <laine@redhat.com>
Date: Fri, 21 Feb 2025 04:24:05 +0000 (-0500)
Subject: tests: be consistent about following DO_TEST_*() with a ;
X-Git-Tag: v11.1.0-rc1~17
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=6f70da39c1fda92bdf5e2f363880da9f4c4ba90e;p=thirdparty%2Flibvirt.git

tests: be consistent about following DO_TEST_*() with a ;

As is often the case with macros (especially those that resolve to
multiple statements), it isn't technically necessary to end any of the
invocations of the DO_TEST_*() macros with a semicolon (as evidenced
by the lines changed in this path). Having does make some
auto-indenters (e.g. cc-mode in emacs) more likely to do the right
thing, though, and it also looks nicer if all the lines are similar.

Signed-off-by: Laine Stump <laine@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
---

diff --git a/tests/qemuxmlconftest.c b/tests/qemuxmlconftest.c
index 905b283f94..fdf8dfe614 100644
--- a/tests/qemuxmlconftest.c
+++ b/tests/qemuxmlconftest.c
@@ -3019,15 +3019,15 @@ mymain(void)
     DO_TEST_CAPS_LATEST("tap-vhost-incorrect");
     DO_TEST_CAPS_LATEST("tap-vhost");
 
-    DO_TEST_CAPS_LATEST("mtp-usb-device")
-    DO_TEST_CAPS_LATEST("net-usb")
-    DO_TEST_CAPS_LATEST("sound-device-virtio")
-    DO_TEST_CAPS_LATEST("pstore-acpi-erst")
-
-    DO_TEST_CAPS_LATEST_FAILURE("disk-network-iscsi-zero-hosts-invalid")
-    DO_TEST_CAPS_LATEST_PARSE_ERROR("hostdev-scsi-vhost-rawio-invalid")
-    DO_TEST_CAPS_LATEST_PARSE_ERROR("hostdev-scsi-vhost-sgio-invalid")
-    DO_TEST_CAPS_LATEST("disk-startuppolicy-optional-drop")
+    DO_TEST_CAPS_LATEST("mtp-usb-device");
+    DO_TEST_CAPS_LATEST("net-usb");
+    DO_TEST_CAPS_LATEST("sound-device-virtio");
+    DO_TEST_CAPS_LATEST("pstore-acpi-erst");
+
+    DO_TEST_CAPS_LATEST_FAILURE("disk-network-iscsi-zero-hosts-invalid");
+    DO_TEST_CAPS_LATEST_PARSE_ERROR("hostdev-scsi-vhost-rawio-invalid");
+    DO_TEST_CAPS_LATEST_PARSE_ERROR("hostdev-scsi-vhost-sgio-invalid");
+    DO_TEST_CAPS_LATEST("disk-startuppolicy-optional-drop");
 
     /* check that all input files were actually used here */
     if (testConfXMLCheck(existingTestCases) < 0)