"Daniel P. Berrange" <berrange@xxxxxxxxxx> wrote: > This is a repost of the patch from: > > http://www.redhat.com/archives/libvir-list/2009-January/msg00583.html > > updated to take account of the changes commited since I first posted it. Thanks. The fix looks right. Glad you added those tests. It's nice to have all of those constants documented, too. ACK. Since DO_TEST is now a trivial specialization of the new DO_TEST_MIGRATE, you can simplify. Plus, I added a "const". >From f48aa04fe5cef250f7f505fecb3d101b87262481 Mon Sep 17 00:00:00 2001 From: Jim Meyering <meyering@xxxxxxxxxx> Date: Thu, 29 Jan 2009 14:39:43 +0100 Subject: [PATCH] DO_TEST: rewrite to use new DO_TEST_MIGRATE --- tests/qemuxml2argvtest.c | 9 ++------- 1 files changed, 2 insertions(+), 7 deletions(-) diff --git a/tests/qemuxml2argvtest.c b/tests/qemuxml2argvtest.c index bbd04fc..0be9ac3 100644 --- a/tests/qemuxml2argvtest.c +++ b/tests/qemuxml2argvtest.c @@ -162,16 +162,11 @@ mymain(int argc, char **argv) return EXIT_FAILURE; #define DO_TEST(name, extraFlags) \ - do { \ - struct testInfo info = { name, extraFlags, NULL }; \ - if (virtTestRun("QEMU XML-2-ARGV " name, \ - 1, testCompareXMLToArgvHelper, &info) < 0) \ - ret = -1; \ - } while (0) + DO_TEST_MIGRATE(name, extraFlags, NULL) #define DO_TEST_MIGRATE(name, extraFlags, migrateFrom) \ do { \ - struct testInfo info = { name, extraFlags, migrateFrom }; \ + const struct testInfo info = { name, extraFlags, migrateFrom }; \ if (virtTestRun("QEMU XML-2-ARGV " name, \ 1, testCompareXMLToArgvHelper, &info) < 0) \ ret = -1; \ -- 1.6.1.1.423.gc2891 -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list