[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#177369: xfs: blanks added to end of syslog lines -- wasteful



On Tue, Jan 21, 2003 at 01:20:17PM -0500, Branden Robinson scrawled:
> On Sun, Jan 19, 2003 at 07:09:57AM +0800, Dan Jacobson wrote:
> > wasteful blanks found:
> > 
> > for i in ' ' '  ' '   ';do echo '|||||||||';grep "$i"\$ /var/log/syslog.0;done|\
> > grep xfs|sort -u;false
> > Jan 18 16:27:08 debian xfs[456]: terminating 
> > Jan 19 05:43:52 debian xfs: ignoring font path element /usr/lib/X11/fonts/cyrillic/ (unreadable) 
> > Jan 19 05:43:53 debian xfs: ignoring font path element /usr/lib/X11/fonts/CID (unreadable) 
> 
> If you, or someone reading this report on debian-x, would like to get
> into XFree86 source hacking in a gentle way, here's a pointer to where
> and what needs to be fixed:
> 
> [0] branden@redwald:~/packages/xfree86 % grep -r terminating xc/programs/xfs/*
> xc/programs/xfs/difs/dispatch.c:                NoticeF("terminating\n");
> [0] branden@redwald:~/packages/xfree86 % grep -r unreadable xc/programs/xfs/*
> xc/programs/xfs/difs/fonts.c:           NoticeF("ignoring font path element %s (unreadable)\n", fpe->name);
> 
> My guess is those explicit newlines don't need to be passed to
> NoticeF().
> 
> Extra points for merging a patch for the above into patch #003.

203 occurrences between NoticeF, ErrorF, Error and FatalError. No
thanks.

-- 
Daniel Stone                                     <dstone@trinity.unimelb.edu.au>
Developer, Trinity College, University of Melbourne

Attachment: pgpubjlFTmBRi.pgp
Description: PGP signature


Reply to: