xfs-masters
[Top] [All Lists]

[xfs-masters] Re: [PATCH] xfs: add newline in debug messages

To: xfs-masters@xxxxxxxxxxx
Subject: [xfs-masters] Re: [PATCH] xfs: add newline in debug messages
From: Eric Sandeen <sandeen@xxxxxxxxxxx>
Date: Sat, 30 Dec 2006 12:40:30 -0600
Cc: xfs@xxxxxxxxxxx, devel@xxxxxxxxxx
In-reply-to: <20061229151811.GA11120@xxxxxxxxxxxxxxx>
References: <20061229151811.GA11120@xxxxxxxxxxxxxxx>
Reply-to: xfs-masters@xxxxxxxxxxx
Sender: xfs-masters-bounce@xxxxxxxxxxx
User-agent: Thunderbird 1.5.0.9 (Macintosh/20061207)
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


<Prev in Thread] Current Thread [Next in Thread>