[PATCH] added missing '\n' to ldb_debug messages
Andrew Bartlett
abartlet at samba.org
Thu Jul 9 19:52:24 MDT 2009
On Thu, 2009-07-09 at 17:23 +0200, Sumit Bose wrote:
> Hi,
>
> this patch adds a few missing '\n' to ldb debug messages to make the
> debug experience more consistent.
I would rather the reverse - that we never specify the \n.
The inclusion of \n in Samba's DEBUG() messages is a hack dating back
years, to do with the way Samba outputs multiple line messages. I don't
think we need to infect ldb with that.
Instead, ldb_debug_stderr() should be patched to output a \n on the end
of the line, as the log format requires. (This message can be
redirected, and may not go to a log, and so should not have the log
formatting oddity embeded in the code).
Andrew Bartlett
--
Andrew Bartlett
http://samba.org/~abartlet/
Authentication Developer, Samba Team http://samba.org
Samba Developer, Cisco Inc.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : http://lists.samba.org/archive/samba-technical/attachments/20090710/70a7a865/attachment.bin
More information about the samba-technical
mailing list