diff options
| author | Tor Lillqvist <tml@collabora.com> | 2016-02-18 09:59:38 +0200 |
|---|---|---|
| committer | Tor Lillqvist <tml@collabora.com> | 2016-02-18 10:04:41 +0200 |
| commit | bd707a4831f6ad0a343729cc057a0daf866b0333 (patch) | |
| tree | e6e626c9dc89f5cc1a46e8a9e7e0421d9a03e0a7 | |
| parent | a6e8910a3c5d33e671a13559438b7228596b8bca (diff) | |
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
| -rw-r--r-- | vcl/source/window/toolbox.cxx | 4 |
1 files 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; } |