diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-01-22 15:52:16 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-01-24 07:56:01 +0100 |
| commit | de8f6b25de6fbe813fe172542e7eff1596b37335 (patch) | |
| tree | 4a2864c87395463391cd2ad40c4f1ada962f44e9 /sc/source | |
| parent | 182a3c7e12a0f56d664deaf67d17bc51eef6299d (diff) | |
loplugin:unused-returns in vcl
Change-Id: I507320900a47f604d17ed7d402d531a7cbdf744e
Reviewed-on: https://gerrit.libreoffice.org/48331
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sc/source')
| -rw-r--r-- | sc/source/ui/inc/select.hxx | 4 | ||||
| -rw-r--r-- | sc/source/ui/view/select.cxx | 14 |
2 files changed, 8 insertions, 10 deletions
diff --git a/sc/source/ui/inc/select.hxx b/sc/source/ui/inc/select.hxx index b78538ca4592..0a2f37003956 100644 --- a/sc/source/ui/inc/select.hxx +++ b/sc/source/ui/inc/select.hxx @@ -65,7 +65,7 @@ public: virtual void BeginDrag() override; virtual void CreateAnchor() override; virtual void DestroyAnchor() override; - virtual bool SetCursorAtPoint( const Point& rPointPixel, bool bDontSelectAtCursor = false ) override; + virtual void SetCursorAtPoint( const Point& rPointPixel, bool bDontSelectAtCursor = false ) override; virtual bool IsSelectionAtPoint( const Point& rPointPixel ) override; virtual void DeselectAtPoint( const Point& rPointPixel ) override; virtual void DeselectAll() override; @@ -92,7 +92,7 @@ public: virtual void BeginDrag() override; virtual void CreateAnchor() override; virtual void DestroyAnchor() override; - virtual bool SetCursorAtPoint( const Point& rPointPixel, bool bDontSelectAtCursor = false ) override; + virtual void SetCursorAtPoint( const Point& rPointPixel, bool bDontSelectAtCursor = false ) override; virtual bool IsSelectionAtPoint( const Point& rPointPixel ) override; virtual void DeselectAtPoint( const Point& rPointPixel ) override; virtual void DeselectAll() override; diff --git a/sc/source/ui/view/select.cxx b/sc/source/ui/view/select.cxx index 34edfe50b2c7..a7da3ed68196 100644 --- a/sc/source/ui/view/select.cxx +++ b/sc/source/ui/view/select.cxx @@ -286,12 +286,12 @@ void ScViewFunctionSet::SetAnchorFlag( bool bSet ) bAnchor = bSet; } -bool ScViewFunctionSet::SetCursorAtPoint( const Point& rPointPixel, bool /* bDontSelectAtCursor */ ) +void ScViewFunctionSet::SetCursorAtPoint( const Point& rPointPixel, bool /* bDontSelectAtCursor */ ) { if ( bDidSwitch ) { if ( rPointPixel == aSwitchPos ) - return false; // don't scroll in wrong window + return; // don't scroll in wrong window else bDidSwitch = false; } @@ -400,7 +400,7 @@ bool ScViewFunctionSet::SetCursorAtPoint( const Point& rPointPixel, bool /* bDon } pViewData->ResetOldCursor(); - return SetCursorAtCell( nPosX, nPosY, bScroll ); + SetCursorAtCell( nPosX, nPosY, bScroll ); } bool ScViewFunctionSet::SetCursorAtCell( SCCOL nPosX, SCROW nPosY, bool bScroll ) @@ -810,13 +810,13 @@ void ScHeaderFunctionSet::DestroyAnchor() bAnchor = false; } -bool ScHeaderFunctionSet::SetCursorAtPoint( const Point& rPointPixel, bool /* bDontSelectAtCursor */ ) +void ScHeaderFunctionSet::SetCursorAtPoint( const Point& rPointPixel, bool /* bDontSelectAtCursor */ ) { if ( bDidSwitch ) { // next valid position has to be originated from another window if ( rPointPixel == aSwitchPos ) - return false; // don't scroll in the wrong window + return; // don't scroll in the wrong window else bDidSwitch = false; } @@ -873,7 +873,7 @@ bool ScHeaderFunctionSet::SetCursorAtPoint( const Point& rPointPixel, bool /* bD { aSwitchPos = rPointPixel; bDidSwitch = true; - return false; // do not crunch with wrong positions + return; // do not crunch with wrong positions } SCCOL nPosX; @@ -917,8 +917,6 @@ bool ScHeaderFunctionSet::SetCursorAtPoint( const Point& rPointPixel, bool /* bD if (bHide) pView->ShowAllCursors(); - - return true; } bool ScHeaderFunctionSet::IsSelectionAtPoint( const Point& rPointPixel ) |