[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cygwin-apps
Subject:    Re: [PATCH] Incidental setup.exe patches #2: Fix chooser vertical  scroll  bar.
From:       Dave Korn <dave.korn.cygwin () googlemail ! com>
Date:       2010-04-14 2:56:38
Message-ID: 4BC52EE6.6010603 () gmail ! com
[Download RAW message or body]

On 13/04/2010 19:08, Dave Korn wrote:

>   Because I was everso disciplined 

  Sigh, and just to remind me that you should never gloat on the internet, I
left a line of debug trace in there.  Doesn't impair the operation in any way,
but it could leave a lot of clutter in the logs.

	* PickView.cc (PickView::WindowProc): Delete inadvertently-committed
	debug logging.

  Committed as obvious, after testing in the debugger to make sure I would
actually see all the logging going on this time.

    cheers,
      DaveK

["oops.diff" (text/x-c)]

Index: PickView.cc
===================================================================
RCS file: /cvs/cygwin-apps/setup/PickView.cc,v
retrieving revision 2.37
diff -p -u -r2.37 PickView.cc
--- PickView.cc	13 Apr 2010 17:49:11 -0000	2.37
+++ PickView.cc	14 Apr 2010 02:35:44 -0000
@@ -818,8 +818,6 @@ PickView::WindowProc (UINT message, WPAR
       {
         // Note: WM_SIZE msgs only appear when 'just' scrolling the window
         RECT windowRect = GetWindowRect ();
-	log(LOG_BABBLE) << "WR: (" << windowRect.left << "," << windowRect.top
-		<< ") x (" << windowRect.right << "," << windowRect.bottom << ")" << endLog;
         if (hasWindowRect)
           {
             int dx;


[prev in list] [next in list] [prev in thread] [next in thread]