Re: [libvirt] [PATCH 1/9] s/int/virLogPriority/ in logging code

On 09/27/2012 12:18 PM, Eric Blake wrote: Spoke too soon; I got a test failure: testutils.c: In function 'virtTestMain': testutils.c:613:32: error: passing argument 1 of 'virLogDefineOutput' from incompatible pointer type [-Werror] In file included from testutils.c:47:0: ../src/util/logging.h:127:12: note: expected 'virLogOutputFunc' but argument is of type 'void (*)(const char *, int, const char *, long long int, const char *, unsigned int, const char *, const char *, void *)' Squash this in: diff --git i/tests/testutils.c w/tests/testutils.c index a33b10d..86be3f5 100644 --- i/tests/testutils.c +++ w/tests/testutils.c @@ -481,7 +481,7 @@ static struct virtTestLogData testLog = { VIR_BUFFER_INITIALIZER }; static void virtTestLogOutput(const char *category ATTRIBUTE_UNUSED, - int priority ATTRIBUTE_UNUSED, + virLogPriority priority ATTRIBUTE_UNUSED, const char *funcname ATTRIBUTE_UNUSED, long long lineno ATTRIBUTE_UNUSED, const char *timestamp, @@ -610,7 +610,7 @@ int virtTestMain(int argc, virLogSetFromEnv(); if (!getenv("LIBVIRT_DEBUG") && !virLogGetNbOutputs()) { if (virLogDefineOutput(virtTestLogOutput, virtTestLogClose, &testLog, - 0, 0, NULL, 0) < 0) + VIR_LOG_DEBUG, 0, NULL, 0) < 0) return 1; } -- Eric Blake eblake(a)redhat.com +1-919-301-3266 Libvirt virtualization library http://libvirt.org