[Libreoffice-commits] core.git: vcl/source

Tor Lillqvist tml at collabora.com
Thu Feb 18 08:06:24 UTC 2016


 vcl/source/window/toolbox.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit bd707a4831f6ad0a343729cc057a0daf866b0333
Author: Tor Lillqvist <tml at collabora.com>
Date:   Thu Feb 18 09:59:38 2016 +0200

    WaE: not necessary to pass this argument, it defaults to the same value
    
    [loplugin:defaultparams] I strongly dislike default parameters. And
    confusing boolean parameters. Here what I really would prefer would be
    to have two separate functions, InvalidateUpperSpin() and
    InvalidateLowerSpin() (or something like that), neither taking any
    parameter at all.
    
    Change-Id: I42c92339481cf576397822c318db1449a576a06a

diff --git a/vcl/source/window/toolbox.cxx b/vcl/source/window/toolbox.cxx
index 0d076a4..e65d04f 100644
--- a/vcl/source/window/toolbox.cxx
+++ b/vcl/source/window/toolbox.cxx
@@ -3468,7 +3468,7 @@ bool ToolBox::ImplHandleMouseMove( const MouseEvent& rMEvt, bool bRepeat )
         if ( bNewIn != mbIn )
         {
             mbIn = bNewIn;
-            InvalidateSpin(false, true);
+            InvalidateSpin(false);
         }
         return true;
     }
@@ -3988,7 +3988,7 @@ void ToolBox::MouseButtonDown( const MouseEvent& rMEvt )
                 StartTracking();
                 mbLower = true;
                 mbIn    = true;
-                InvalidateSpin(false, true);
+                InvalidateSpin(false);
             }
             return;
         }


More information about the Libreoffice-commits mailing list