| 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@localhost.sw.ru> |
| References: | <20061229151811.GA11120@localhost.sw.ru> |
| 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> |
|---|---|---|
| ||
| Previous by Date: | [xfs-masters] [PATCH] xfs: add newline in debug messages, Alexey Dobriyan |
|---|---|
| Previous by Thread: | [xfs-masters] [PATCH] xfs: add newline in debug messages, Alexey Dobriyan |
| Indexes: | [Date] [Thread] [Top] [All Lists] |