diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-06-26 11:44:49 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-06-26 14:00:59 +0200 |
| commit | 2fb2d6efdb25f718ddd65709697bc68d81815a59 (patch) | |
| tree | a73b80fb06b773204613b1446256355baea01b51 /sdext | |
| parent | 3b9d2501b390dfaec7d0a189742fb3305f3742d9 (diff) | |
loplugin:oncevar in scaddins..sot
Change-Id: I657a131374b9f1153135e00ed429889518b91427
Reviewed-on: https://gerrit.libreoffice.org/39258
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sdext')
| -rw-r--r-- | sdext/source/minimizer/configurationaccess.cxx | 4 | ||||
| -rw-r--r-- | sdext/source/presenter/PresenterAccessibility.cxx | 3 | ||||
| -rw-r--r-- | sdext/source/presenter/PresenterWindowManager.cxx | 4 |
3 files changed, 3 insertions, 8 deletions
diff --git a/sdext/source/minimizer/configurationaccess.cxx b/sdext/source/minimizer/configurationaccess.cxx index 2206093bc1f7..11e3e8a6d034 100644 --- a/sdext/source/minimizer/configurationaccess.cxx +++ b/sdext/source/minimizer/configurationaccess.cxx @@ -113,9 +113,7 @@ void OptimizerSettings::SaveSettingsToConfiguration( const Reference< XNameRepla // Any( maFilterName ), Any( mbOpenNewDocument ) }; - sal_Int32 i, nCount = SAL_N_ELEMENTS( pNames ); - - for ( i = 0; i < nCount; i++ ) + for ( int i = 0; i < int(SAL_N_ELEMENTS( pNames )); i++ ) { try { diff --git a/sdext/source/presenter/PresenterAccessibility.cxx b/sdext/source/presenter/PresenterAccessibility.cxx index 3dd042d6cc79..93392a74c1c1 100644 --- a/sdext/source/presenter/PresenterAccessibility.cxx +++ b/sdext/source/presenter/PresenterAccessibility.cxx @@ -1492,8 +1492,7 @@ sal_Int32 SAL_CALL PresenterAccessible::AccessibleParagraph::getIndexAtPoint ( const css::awt::Point& ) { ThrowIfDisposed(); - sal_Int32 nIndex (-1); - return nIndex; + return -1; } OUString SAL_CALL PresenterAccessible::AccessibleParagraph::getSelectedText() diff --git a/sdext/source/presenter/PresenterWindowManager.cxx b/sdext/source/presenter/PresenterWindowManager.cxx index f9de66cf2920..70687aac7afb 100644 --- a/sdext/source/presenter/PresenterWindowManager.cxx +++ b/sdext/source/presenter/PresenterWindowManager.cxx @@ -330,8 +330,6 @@ void SAL_CALL PresenterWindowManager::disposing (const lang::EventObject& rEvent bool PresenterWindowManager::PaintChildren (const awt::PaintEvent& rEvent) const { - bool bChildInvalidated (false); - // Call windowPaint on all children that lie in or touch the // update rectangle. PresenterPaneContainer::PaneList::const_iterator iPane; @@ -381,7 +379,7 @@ bool PresenterWindowManager::PaintChildren (const awt::PaintEvent& rEvent) const } } - return bChildInvalidated; + return false; } void PresenterWindowManager::SetLayoutMode (const LayoutMode eMode) |