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
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
