On 05/11/2011 02:31 PM, Jiri Denemark wrote: > On Wed, May 11, 2011 at 14:13:28 -0600, Eric Blake wrote: >> Use of ',##__VA_ARGS__' is a gcc extension not guaranteed by >> C99; thankfully, we can avoid it by lumping the format argument >> into the var-args set. >> >> * src/util/logging.h (VIR_DEBUG_INT, VIR_INFO_INT, VIR_WARN_INT) >> (VIR_ERROR_INT, VIR_DEBUG, VIR_INFO, VIR_WARN, VIR_ERROR): Stick >> to C99 var-arg macro syntax. >> * examples/domain-events/events-c/event-test.c (VIR_DEBUG): >> Simplify. >> --- >> >> This one turned out to be much simpler than I was fearing; any >> time we guarantee that (fmt,...) expands to (fmt,##__VA_ARGS__), >> then we just use one fewer argument (...) expanding to (__VA_ARGS__). >> >> examples/domain-events/events-c/event-test.c | 3 +- >> src/util/logging.h | 34 +++++++++++++------------- >> 2 files changed, 18 insertions(+), 19 deletions(-) > > Nice and simple, ACK. Thanks; I've pushed Lai's cleanup and my followup now. The remaining two patches in this series are still worthwhile cleanups, but as of this patch we're back to the state of no longer relying on gcc extensions. -- Eric Blake eblake@xxxxxxxxxx +1-801-349-2682 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list