From c5155f59e9fb7f5841de8a6a6fae2219400ac508 Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Mon, 13 Aug 2018 11:21:28 +0200 Subject: loplugin:returnconstant in SwEditShell Change-Id: I0daa8938e81017f8c6db4da55aed297f767799b7 Reviewed-on: https://gerrit.libreoffice.org/58959 Tested-by: Jenkins Reviewed-by: Noel Grandin --- sw/source/core/edit/edtox.cxx | 3 +-- sw/source/uibase/wrtsh/wrtsh2.cxx | 6 ++---- 2 files changed, 3 insertions(+), 6 deletions(-) (limited to 'sw/source') diff --git a/sw/source/core/edit/edtox.cxx b/sw/source/core/edit/edtox.cxx index 3a9f0f94c7d2..80526cc81b5f 100644 --- a/sw/source/core/edit/edtox.cxx +++ b/sw/source/core/edit/edtox.cxx @@ -154,7 +154,7 @@ void SwEditShell::InsertTableOf( const SwTOXBase& rTOX, const SfxItemSet* pSet ) } /// update tables of content -bool SwEditShell::UpdateTableOf( const SwTOXBase& rTOX, const SfxItemSet* pSet ) +void SwEditShell::UpdateTableOf( const SwTOXBase& rTOX, const SfxItemSet* pSet ) { OSL_ENSURE( dynamic_cast( &rTOX) != nullptr, "no TOXBaseSection!" ); SwTOXBaseSection* pTOX = const_cast(static_cast(&rTOX)); @@ -191,7 +191,6 @@ bool SwEditShell::UpdateTableOf( const SwTOXBase& rTOX, const SfxItemSet* pSet ) ::EndProgress( pDocSh ); EndAllAction(); } - return false; } /// Get current listing before or at the Cursor diff --git a/sw/source/uibase/wrtsh/wrtsh2.cxx b/sw/source/uibase/wrtsh/wrtsh2.cxx index ecf9a023995d..1dfaa712ecfe 100644 --- a/sw/source/uibase/wrtsh/wrtsh2.cxx +++ b/sw/source/uibase/wrtsh/wrtsh2.cxx @@ -320,11 +320,9 @@ void SwWrtShell::InsertTableOf(const SwTOXBase& rTOX, const SfxItemSet* pSet) bool SwWrtShell::UpdateTableOf(const SwTOXBase& rTOX, const SfxItemSet* pSet) { - bool bResult = false; - if(CanInsert()) { - bResult = SwEditShell::UpdateTableOf(rTOX, pSet); + SwEditShell::UpdateTableOf(rTOX, pSet); if (pSet == nullptr) { @@ -336,7 +334,7 @@ bool SwWrtShell::UpdateTableOf(const SwTOXBase& rTOX, const SfxItemSet* pSet) } } - return bResult; + return false; } // handler for click on the field given as parameter. -- cgit