Adam Martinson <[email protected]> writes:

>  #define U(str) BAD_CAST str
> -#define DBG(str) wine_dbgstr_a((char const*)str)
> +#define DBG(args...) wine_dbg_sprintf((char const*)args)
> +#define RULE(name, args...) TRACE("Got "#name": %s\n", DBG(args))

Again, you can't use varargs macros.

You have a tendency to abuse macros, please try to avoid them as much as
possible. In this case, replacing a standard TRACE by a file-specific
macro is not an improvement, even if it saves a little typing. People
must be able to understand your code without having to look up a ton of
obscure #defines.

-- 
Alexandre Julliard
[email protected]


Reply via email to