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