[setup PATCH] Remove pointless, worrying log message

Christopher Faylor cgf@redhat.com
Sun Jan 19 17:08:00 GMT 2003


On Sun, Jan 19, 2003 at 03:25:52PM -0000, Max Bowsher wrote:
>Index: compress_bz.cc
>===================================================================
>RCS file: /home/max/cvsmirror/cygwin-apps-cvs/setup/compress_bz.cc,v
>retrieving revision 2.7
>diff -u -r2.7 compress_bz.cc
>--- compress_bz.cc 10 Nov 2002 03:35:06 -0000 2.7
>+++ compress_bz.cc 19 Jan 2003 15:21:14 -0000
>@@ -194,7 +194,6 @@
> int
> compress_bz::error ()
> {
>-  log (LOG_TIMESTAMP, "compress_bz::error called");
>   return 0;
> }
>
>Code calls compress_bz::error() to ask *whether* an error has occurred. It
>is silly and scare-mongering to print "compress_bz::error called" to the log
>in this case.
>
>Ok to commit?

I don't want to override Rob here, but IMO that's a big YES.

cgf



More information about the Cygwin-apps mailing list