Question

I have a logging macro which in release mode becomes:

#define LOG (void)

So statement

LOG("foobar %d", 0xbabecafe);

is expanded to

(void)("foobar %d", 0xbabecafe);

The problem is that the last expression produces an warning under gcc:

warning: left-hand operand of comma expression has no effect [-Wunused-value]

How can I change the logging macro such that no warning is issued? (Note, that I don’t want to add compiling flag -Wunused-value).

EDIT I see already a couple of answers involving (...). The same file is compiled under Minix which doesn’t support variadic macros. The best would be to have a C89 conforming solution. While your answer is correct (and I upvoted it), it is my fault that I didn’t include this small detail.

Answer 1

I think the old school way of dealing with this is to take advantage of double parens. Something like this:

LOG(("message: %d", 10));

Then for your macro, you define it like this:

#define LOG(x) printf x

or

#define LOG(x) (void)0

Because of the double parens, the pre-processor treats the whole inner paren as a single parameter. This at least used to work in visual studio.

EDIT: I did a quick test, it works with gcc with -ansi, so it should be good:

gcc -DNDEBUG -ansi -pedantic -W -Wall test.c -o test

#include <stdio.h>

#ifdef NDEBUG
#define LOG(x) printf x
#else
#define LOG(x) (void)0
#endif

int main() {
    LOG(("message: %d\n", 10));
    return 0;
}