From 6f70da39c1fda92bdf5e2f363880da9f4c4ba90e Mon Sep 17 00:00:00 2001 From: Laine Stump Date: Thu, 20 Feb 2025 23:24:05 -0500 Subject: [PATCH] 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 Reviewed-by: Michal Privoznik --- tests/qemuxmlconftest.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) 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) -- 2.47.3