[SSSD] [PATCH 3/4] Make DEBUG macro variadic

Simo Sorce simo at redhat.com
Thu Oct 10 20:27:20 UTC 2013


On Thu, 2013-10-10 at 15:46 -0400, Stephen Gallagher wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> On 10/10/2013 10:07 AM, Nikolai Kondrashov wrote:
> > Make DEBUG macro accept variable number of arguments, thus removing
> > the need to wrap the format string and its arguments into parens
> > on invocation.
> 
> Nack.
> 
> Please do us a favor and split this into two patches, one that makes
> the change to the macro definition and one that bulk-updates the
> existing messages. Having a 2.2MB patch file with a few small changes
> buried in it somewhere is very difficult to review. I'm not confident
> I can locate the real changes amidst the find-replace noise.

I would not want to consider such a patch in any case. The churn is just
too extraordinarily big and I do not see enough value to warrant it.

The patch is so immense that is it impossible to review, so we would
have to make a leap of faith in assuming every single change did not
introduce errors.

I am not against 'simplifying' the debug macro if possible, but then it
should probably be done in a backwards compatible way and debug
statements changed in time, just like we did when we switched from mere
numbers to error type definitions.

For example we could call the new macro DBG, an start using it in place
of DEBUG in new code. DEBUG would be just a wrapper around DBG that
maintains the old interface.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York




More information about the sssd-devel mailing list