[PATCH 1/2] tests: be consistent about following DO_TEST_*() with a ;

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

 



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@xxxxxxxxxx>
---
 tests/qemuxmlconftest.c | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/tests/qemuxmlconftest.c b/tests/qemuxmlconftest.c
index c271170d25..13a8577aad 100644
--- a/tests/qemuxmlconftest.c
+++ b/tests/qemuxmlconftest.c
@@ -3017,15 +3017,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)
-- 
2.48.1




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux