Re: [xfs-masters] [PATCH] xfs: add newline in debug messages [message #9397 is a reply to message #9393] |
Sat, 30 December 2006 18:40  |
Eric Sandeen
Messages: 4 Registered: September 2006
|
Junior Member |
|
|
Alexey Dobriyan wrote:
> cmn_err(CE_DEBUG, ...); doesn't add newline for you.
The patch looks about right to me. This -is- how it was -intended- to
be used.
However, the no-newline behavior is so rarely used that I almost wonder
if it'd be more straightforward to drop it, and instead use sprintf or
something to construct those strings... otherwise I'm sure that future
CE_DEBUG messages will forget newlines as well.
-Eric
|
|
|