On 10/20/2010 03:14 AM, Matthias Bolte wrote:
Hmm - ATTRIBUTE_UNUSED is an indication that on some paths, the variable might not be used; and not a requirement that it must not be used. Would anyone object to this followup, for one less #ifdef? diff --git i/src/util/virtaudit.c w/src/util/virtaudit.c index 965a6e8..3226961 100644 --- i/src/util/virtaudit.c +++ w/src/util/virtaudit.c @@ -71,18 +71,12 @@ void virAuditLog(int logging) } -#if HAVE_AUDIT -void virAuditSend(const char *file ATTRIBUTE_UNUSED, const char *func, size_t linenr, - const char *clienttty, const char *clientaddr, - enum virAuditRecordType type, bool success, - const char *fmt, ...) -#else -void virAuditSend(const char *file ATTRIBUTE_UNUSED, const char *func, size_t linenr, +void virAuditSend(const char *file ATTRIBUTE_UNUSED, const char *func, + size_t linenr, const char *clienttty ATTRIBUTE_UNUSED, const char *clientaddr ATTRIBUTE_UNUSED, enum virAuditRecordType type ATTRIBUTE_UNUSED, bool success, const char *fmt, ...) -#endif { char *str = NULL; va_list args;ACK.
Thanks; applied. -- Eric Blake eblake@xxxxxxxxxx +1-801-349-2682 Libvirt virtualization library http://libvirt.org -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list