[SSSD] DEBUG macro refactoring v3

Nikolai Kondrashov Nikolai.Kondrashov at redhat.com
Tue Jan 7 14:59:26 UTC 2014


On 01/07/2014 04:53 PM, Jakub Hrozek wrote:
> On Tue, Jan 07, 2014 at 03:31:50PM +0200, Nikolai Kondrashov wrote:
>> On 01/07/2014 02:33 PM, Jakub Hrozek wrote:
>>> [PATCH 3/8] Remove extra flushing from debug message output
>>> The code is fine, but why is it a separate patch and not part of #2 ? It
>>> seems the issue was introduced in patch 2/8, right?
>>
>> No, it was there before my changes, in the original "debug_fn" function, which
>> body was moved to the new "debug_vprintf". That's why I kept it separate.
>> However, please squash it, if you think it's right.
>
> Ah, sorry, my review was flawed. You're right.
>
> Nevertheless, I think I found one bug in patch #2 when testing Stephen's
> follow-up patches. I don't think it's correct to replace all occurences
> of DEBUG_MSG with debug_fn. The reason is that DEBUG_MSG used to perform
> check if a particular level is set, debug_fn prints directly.
>
> One example of where we need to check the debug level is
> ldb_debug_messages(), with the current version of the patch, the ldb
> messages are always printed.

You're right. This is a regression. I forgot to update that when I moved the
level limiting back to the macro. Will fix.

Sincerely,
Nick



More information about the sssd-devel mailing list