From bd707a4831f6ad0a343729cc057a0daf866b0333 Mon Sep 17 00:00:00 2001 From: Tor Lillqvist Date: Thu, 18 Feb 2016 09:59:38 +0200 Subject: 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 --- vcl/source/window/toolbox.cxx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/vcl/source/window/toolbox.cxx b/vcl/source/window/toolbox.cxx index 0d076a498598..e65d04f96773 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; } -- cgit