[Home]

Summary:ASTERISK-12334: [patch] spaces after newlines in logging
Reporter:Peter Grayson (jpgrayson)Labels:
Date Opened:2008-07-07 15:21:29Date Closed:2008-07-07 15:40:30
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Channels/chan_iax2
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) chan_iax2_newline_after_func.patch
( 1) chan_iax2_space_after_newline.patch
( 2) chan_iax2_spelling.patch
Description:In chan_iax2, there are two log messages that have a superfluous space character after the newline: "blah\n ".

This causes the subsequent log message to start in column 1 instead of column 0.
Comments:By: Peter Grayson (jpgrayson) 2008-07-07 15:25:08

I added a second trivial whitespace patch that simply adds a newline character after the closing brace of some function definitions.

int foo()
{
 ...
}
int bar()
{
 ...
}

becomes

int foo()
{
 ...
}

int bar()
{
 ...
}
By: Peter Grayson (jpgrayson) 2008-07-07 15:29:49

Yet another small patch, this one fixes an obvious spelling problem.
By: Digium Subversion (svnbot) 2008-07-07 15:40:27

Repository: asterisk
Revision: 128737

U   branches/1.4/channels/chan_iax2.c

------------------------------------------------------------------------
r128737 | seanbright | 2008-07-07 15:40:24 -0500 (Mon, 07 Jul 2008) | 9 lines

Remove spurious trailing whitespace from log messages and fix a spelling error
in a log message.

(closes issue ASTERISK-12334)
Reported by: jpgrayson
Patches:
     chan_iax2_space_after_newline.patch uploaded by jpgrayson (license 492)
     chan_iax2_spelling.patch uploaded by jpgrayson (license 492)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=128737