Add missing newline to WARN messages

Francois Gouget fgouget at free.fr
Wed May 9 11:31:17 CDT 2001


On Wed, 9 May 2001, Dmitry Timoshkov wrote:

> "Francois Gouget" <fgouget at free.fr> wrote:
> 
> >  * I scanned the source for TRACE/WARN/ERR/FIXME statements without a
> > terminating newline. There are few cases where this is justified because
> > of following DPRINTFs. But most of the time there should be a \n
> > otherwise the trace will not be flushed to disk immediately and the logs
> > will look bad.
> 
> You forgot to remove redundant ' around %s in the traces when output is made
> by debugstr_a or debugstr_w. Actually you removed some of them, but not all.

   Yes, I was not scanning specifically for that. I may have noticed one
or two and fixed as I went, but I probably fixed very very few (just
one?).
   And it's probably not as bad (it doesn't delay the printing of the
trace). Also the problem is to find a grep that yields a usable list of
places to check.
   Ah. Found something that could work:
   $ wg . "'%s'" | egrep "\) *; *$" | grep debugstr_ | wc -l
   10
   $ wg . "'%s'" | egrep -v "\) *; *$" | wc -l
   142

   (wg is my recursive grep script)
   But it's nice to see you volunteer for this work ;-)


--
Francois Gouget         fgouget at free.fr        http://fgouget.free.fr/
                           La terre est une bêta...





More information about the wine-devel mailing list