[Libreoffice-commits] core.git: basctl/source canvas/source cui/source dbaccess/source sc/source slideshow/source sw/inc sw/source
Libreoffice Gerrit user
logerrit at kemper.freedesktop.org
Tue Oct 30 09:57:30 UTC 2018
basctl/source/basicide/baside2.cxx | 6 +++---
basctl/source/basicide/baside2.hxx | 3 +--
basctl/source/basicide/baside2b.cxx | 8 +++-----
basctl/source/basicide/basides1.cxx | 5 ++---
canvas/source/opengl/ogl_spritedevicehelper.cxx | 9 +--------
canvas/source/opengl/ogl_spritedevicehelper.hxx | 2 --
canvas/source/vcl/textlayout.cxx | 4 +---
canvas/source/vcl/textlayout.hxx | 2 +-
cui/source/inc/autocdlg.hxx | 2 +-
cui/source/inc/macroass.hxx | 2 +-
cui/source/tabpages/autocdlg.cxx | 6 +++---
cui/source/tabpages/macroass.cxx | 8 ++++----
dbaccess/source/ui/inc/DExport.hxx | 2 +-
dbaccess/source/ui/misc/DExport.cxx | 5 ++---
sc/source/ui/vba/vbarange.cxx | 3 +--
slideshow/source/engine/shapes/viewshape.cxx | 4 +---
slideshow/source/engine/shapes/viewshape.hxx | 4 +---
sw/inc/editsh.hxx | 6 ++----
sw/source/core/edit/edglss.cxx | 9 +++------
sw/source/uibase/dochdl/gloshdl.cxx | 7 +------
20 files changed, 33 insertions(+), 64 deletions(-)
New commits:
commit a4b60b78ea36d55a2abf0e5efccd8530568d2209
Author: Noel Grandin <noel.grandin at collabora.co.uk>
AuthorDate: Tue Oct 30 09:20:58 2018 +0200
Commit: Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Tue Oct 30 10:57:04 2018 +0100
loplugin:returnconstant
Change-Id: I5b859de6ccd908eee4356acbe1f12b441ab36df3
Reviewed-on: https://gerrit.libreoffice.org/62539
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
diff --git a/basctl/source/basicide/baside2.cxx b/basctl/source/basicide/baside2.cxx
index 15d2715a6973..119f7c6b41a3 100644
--- a/basctl/source/basicide/baside2.cxx
+++ b/basctl/source/basicide/baside2.cxx
@@ -593,7 +593,7 @@ void ModulWindow::ManageBreakPoints()
rBrkWin.Invalidate();
}
-bool ModulWindow::BasicErrorHdl( StarBASIC const * pBasic )
+void ModulWindow::BasicErrorHdl( StarBASIC const * pBasic )
{
GetShell()->GetViewFrame()->ToTop();
@@ -630,11 +630,11 @@ bool ModulWindow::BasicErrorHdl( StarBASIC const * pBasic )
// #i47002#
VclPtr<vcl::Window> pWindow = VCLUnoHelper::GetWindow( xWindow );
if ( !pWindow )
- return false;
+ return;
if ( bMarkError )
m_aXEditorWindow->GetBrkWindow().SetNoMarker();
- return false;
+ return;
}
BasicDebugFlags ModulWindow::BasicBreakHdl()
diff --git a/basctl/source/basicide/baside2.hxx b/basctl/source/basicide/baside2.hxx
index c899f1dfdde6..ab46e7de04d2 100644
--- a/basctl/source/basicide/baside2.hxx
+++ b/basctl/source/basicide/baside2.hxx
@@ -199,7 +199,6 @@ class WatchTreeListBox final : public SvHeaderTabListBox
virtual bool EditingEntry( SvTreeListEntry* pEntry, Selection& rSel ) override;
virtual bool EditedEntry( SvTreeListEntry* pEntry, const OUString& rNewText ) override;
- bool ImplBasicEntryEdited( SvTreeListEntry* pEntry, const OUString& rResult );
SbxBase* ImplGetSBXForEntry( SvTreeListEntry* pEntry, bool& rbArrayElement );
public:
@@ -352,7 +351,7 @@ public:
void UpdateBreakPoint( const BreakPoint& rBrk );
void BasicAddWatch();
- bool BasicErrorHdl( StarBASIC const * pBasic );
+ void BasicErrorHdl( StarBASIC const * pBasic );
BasicDebugFlags BasicBreakHdl();
void AssertValidEditEngine();
diff --git a/basctl/source/basicide/baside2b.cxx b/basctl/source/basicide/baside2b.cxx
index ed4c9b23ba2b..4dc3b0571851 100644
--- a/basctl/source/basicide/baside2b.cxx
+++ b/basctl/source/basicide/baside2b.cxx
@@ -2278,11 +2278,9 @@ bool WatchTreeListBox::EditedEntry( SvTreeListEntry* pEntry, const OUString& rNe
if( cFirst == '\"' && cLast == '\"' )
aResult = aResult.copy( 1, nResultLen - 2 );
- return aResult != aEditingRes && ImplBasicEntryEdited(pEntry, aResult);
-}
+ if (aResult == aEditingRes)
+ return false;
-bool WatchTreeListBox::ImplBasicEntryEdited( SvTreeListEntry* pEntry, const OUString& rResult )
-{
bool bArrayElement;
SbxBase* pSBX = ImplGetSBXForEntry( pEntry, bArrayElement );
@@ -2294,7 +2292,7 @@ bool WatchTreeListBox::ImplBasicEntryEdited( SvTreeListEntry* pEntry, const OUSt
{
// If the type is variable, the conversion of the SBX does not matter,
// else the string is converted.
- pVar->PutStringExt( rResult );
+ pVar->PutStringExt( aResult );
}
}
diff --git a/basctl/source/basicide/basides1.cxx b/basctl/source/basicide/basides1.cxx
index 1159777d9037..97540e7d6707 100644
--- a/basctl/source/basicide/basides1.cxx
+++ b/basctl/source/basicide/basides1.cxx
@@ -1242,11 +1242,10 @@ VclPtr<BaseWindow> Shell::FindWindow(
bool Shell::CallBasicErrorHdl( StarBASIC const * pBasic )
{
- bool bRet = false;
VclPtr<ModulWindow> pModWin = ShowActiveModuleWindow( pBasic );
if ( pModWin )
- bRet = pModWin->BasicErrorHdl( pBasic );
- return bRet;
+ pModWin->BasicErrorHdl( pBasic );
+ return false;
}
BasicDebugFlags Shell::CallBasicBreakHdl( StarBASIC const * pBasic )
diff --git a/canvas/source/opengl/ogl_spritedevicehelper.cxx b/canvas/source/opengl/ogl_spritedevicehelper.cxx
index 82345bd7045a..cbbe93a134e3 100644
--- a/canvas/source/opengl/ogl_spritedevicehelper.cxx
+++ b/canvas/source/opengl/ogl_spritedevicehelper.cxx
@@ -273,8 +273,7 @@ namespace oglcanvas
if( !bIsVisible || !mxContext->isInitialized() || !mpSpriteCanvas )
return false;
- if( !activateWindowContext() )
- return false;
+ mxContext->makeCurrent();
SystemChildWindow* pChildWindow = mxContext->getChildWindow();
const ::Size& rOutputSize = pChildWindow->GetSizePixel();
@@ -503,12 +502,6 @@ namespace oglcanvas
setupUniforms(mnRectangularTwoColorGradientProgram, pColors[0], pColors[1], rTexTransform);
}
- bool SpriteDeviceHelper::activateWindowContext()
- {
- mxContext->makeCurrent();
- return true;
- }
-
namespace
{
diff --git a/canvas/source/opengl/ogl_spritedevicehelper.hxx b/canvas/source/opengl/ogl_spritedevicehelper.hxx
index ddc6e27a3e04..4d3664db6eb4 100644
--- a/canvas/source/opengl/ogl_spritedevicehelper.hxx
+++ b/canvas/source/opengl/ogl_spritedevicehelper.hxx
@@ -112,8 +112,6 @@ namespace oglcanvas
/// Get instance of internal texture cache
TextureCache& getTextureCache() const;
- bool activateWindowContext();
-
private:
/// Pointer to sprite canvas (owner of this helper), needed to create bitmaps
SpriteCanvas* mpSpriteCanvas;
diff --git a/canvas/source/vcl/textlayout.cxx b/canvas/source/vcl/textlayout.cxx
index 1a659f463ab7..18307fce9c0a 100644
--- a/canvas/source/vcl/textlayout.cxx
+++ b/canvas/source/vcl/textlayout.cxx
@@ -321,7 +321,7 @@ namespace vclcanvas
return maText;
}
- bool TextLayout::draw( OutputDevice& rOutDev,
+ void TextLayout::draw( OutputDevice& rOutDev,
const Point& rOutpos,
const rendering::ViewState& viewState,
const rendering::RenderState& renderState ) const
@@ -352,8 +352,6 @@ namespace vclcanvas
::canvas::tools::numeric_cast<sal_uInt16>(maText.StartPosition),
::canvas::tools::numeric_cast<sal_uInt16>(maText.Length) );
}
-
- return true;
}
namespace
diff --git a/canvas/source/vcl/textlayout.hxx b/canvas/source/vcl/textlayout.hxx
index e6a661331101..8f5cd1772a4e 100644
--- a/canvas/source/vcl/textlayout.hxx
+++ b/canvas/source/vcl/textlayout.hxx
@@ -82,7 +82,7 @@ namespace vclcanvas
virtual sal_Bool SAL_CALL supportsService( const OUString& ServiceName ) override;
virtual css::uno::Sequence< OUString > SAL_CALL getSupportedServiceNames() override;
- bool draw( OutputDevice& rOutDev,
+ void draw( OutputDevice& rOutDev,
const Point& rOutpos,
const css::rendering::ViewState& viewState,
const css::rendering::RenderState& renderState ) const;
diff --git a/cui/source/inc/autocdlg.hxx b/cui/source/inc/autocdlg.hxx
index 82a937484aba..d021827ddb55 100644
--- a/cui/source/inc/autocdlg.hxx
+++ b/cui/source/inc/autocdlg.hxx
@@ -326,7 +326,7 @@ private:
DECL_LINK(NewDelActionHdl, AutoCorrEdit&, bool);
DECL_LINK(SelectHdl, ListBox&, void);
DECL_LINK(ModifyHdl, Edit&, void);
- bool NewDelHdl(void const *);
+ void NewDelHdl(void const *);
/// Box filled with new language
void RefillReplaceBoxes(bool bFromReset,
LanguageType eOldLanguage,
diff --git a/cui/source/inc/macroass.hxx b/cui/source/inc/macroass.hxx
index 1ed414b37182..3a37af149639 100644
--- a/cui/source/inc/macroass.hxx
+++ b/cui/source/inc/macroass.hxx
@@ -42,7 +42,7 @@ class SfxMacroTabPage final : public SfxTabPage
DECL_LINK( AssignDeleteHdl_Impl, SvTreeListBox*, bool );
DECL_LINK( AssignDeleteClickHdl_Impl, Button *, void );
- bool AssignDeleteHdl(Control const *);
+ void AssignDeleteHdl(Control const *);
DECL_LINK( TimeOut_Impl, Timer*, void );
std::unique_ptr<SfxMacroTabPage_Impl> mpImpl;
diff --git a/cui/source/tabpages/autocdlg.cxx b/cui/source/tabpages/autocdlg.cxx
index 2a08fa616064..1bc9ffc4b780 100644
--- a/cui/source/tabpages/autocdlg.cxx
+++ b/cui/source/tabpages/autocdlg.cxx
@@ -1652,10 +1652,11 @@ IMPL_LINK(OfaAutocorrExceptPage, NewDelButtonHdl, Button*, pBtn, void)
IMPL_LINK(OfaAutocorrExceptPage, NewDelActionHdl, AutoCorrEdit&, rEdit, bool)
{
- return NewDelHdl(&rEdit);
+ NewDelHdl(&rEdit);
+ return false;
}
-bool OfaAutocorrExceptPage::NewDelHdl(void const * pBtn)
+void OfaAutocorrExceptPage::NewDelHdl(void const * pBtn)
{
if((pBtn == m_pNewAbbrevPB || pBtn == m_pAbbrevED.get() )
&& !m_pAbbrevED->GetText().isEmpty())
@@ -1679,7 +1680,6 @@ bool OfaAutocorrExceptPage::NewDelHdl(void const * pBtn)
m_pDoubleCapsLB->RemoveEntry(m_pDoubleCapsED->GetText());
ModifyHdl(*m_pDoubleCapsED);
}
- return false;
}
IMPL_LINK(OfaAutocorrExceptPage, SelectHdl, ListBox&, rBox, void)
diff --git a/cui/source/tabpages/macroass.cxx b/cui/source/tabpages/macroass.cxx
index 9bd6a00f7d71..8f46be227a44 100644
--- a/cui/source/tabpages/macroass.cxx
+++ b/cui/source/tabpages/macroass.cxx
@@ -307,17 +307,18 @@ IMPL_LINK( SfxMacroTabPage, AssignDeleteClickHdl_Impl, Button*, pBtn, void )
IMPL_LINK( SfxMacroTabPage, AssignDeleteHdl_Impl, SvTreeListBox*, pBtn, bool )
{
- return AssignDeleteHdl(pBtn);
+ AssignDeleteHdl(pBtn);
+ return false;
}
-bool SfxMacroTabPage::AssignDeleteHdl(Control const * pBtn)
+void SfxMacroTabPage::AssignDeleteHdl(Control const * pBtn)
{
SvHeaderTabListBox& rListBox = mpImpl->pEventLB->GetListBox();
SvTreeListEntry* pE = rListBox.FirstSelected();
if( !pE || LISTBOX_ENTRY_NOTFOUND == rListBox.GetModel()->GetAbsPos( pE ) )
{
DBG_ASSERT( pE, "Where does the empty entry come from?" );
- return false;
+ return;
}
const bool bAssEnabled = pBtn != mpImpl->pDeletePB && mpImpl->pAssignPB->IsEnabled();
@@ -351,7 +352,6 @@ bool SfxMacroTabPage::AssignDeleteHdl(Control const * pBtn)
rListBox.SetUpdateMode( true );
EnableButtons();
- return false;
}
IMPL_LINK( SfxMacroTabPage, TimeOut_Impl, Timer*,, void )
diff --git a/dbaccess/source/ui/inc/DExport.hxx b/dbaccess/source/ui/inc/DExport.hxx
index 0224943fbd53..580a46577d79 100644
--- a/dbaccess/source/ui/inc/DExport.hxx
+++ b/dbaccess/source/ui/inc/DExport.hxx
@@ -114,7 +114,7 @@ namespace dbaui
void adjustFormat();
void eraseTokens();
void insertValueIntoColumn();
- bool createRowSet();
+ void createRowSet();
void showErrorDialog(const css::sdbc::SQLException& e);
void ensureFormatter();
diff --git a/dbaccess/source/ui/misc/DExport.cxx b/dbaccess/source/ui/misc/DExport.cxx
index e2f87ec2d979..f070b730f9ab 100644
--- a/dbaccess/source/ui/misc/DExport.cxx
+++ b/dbaccess/source/ui/misc/DExport.cxx
@@ -670,10 +670,9 @@ void ODatabaseExport::CreateDefaultColumn(const OUString& _rColumnName)
m_vDestVector.emplace_back(m_aDestColumns.emplace(aAlias,pField).first);
}
-bool ODatabaseExport::createRowSet()
+void ODatabaseExport::createRowSet()
{
m_pUpdateHelper.reset(new OParameterUpdateHelper(createPreparedStatment(m_xConnection->getMetaData(),m_xTable,m_vColumnPositions)));
- return true;
}
bool ODatabaseExport::executeWizard(const OUString& _rTableName, const Any& _aTextColor, const FontDescriptor& _rFont)
@@ -725,7 +724,7 @@ bool ODatabaseExport::executeWizard(const OUString& _rTableName, const Any& _aTe
bError = true;
if(!bError)
- bError = !createRowSet();
+ createRowSet();
}
catch( const SQLException&)
{
diff --git a/sc/source/ui/vba/vbarange.cxx b/sc/source/ui/vba/vbarange.cxx
index 5d90aec39f2a..3d43be0ba219 100644
--- a/sc/source/ui/vba/vbarange.cxx
+++ b/sc/source/ui/vba/vbarange.cxx
@@ -559,7 +559,7 @@ public:
return nType;
}
- bool setNumberFormat( const OUString& rFormat )
+ void setNumberFormat( const OUString& rFormat )
{
// #163288# treat "General" as "Standard" format
sal_Int32 nNewIndex = 0;
@@ -573,7 +573,6 @@ public:
nNewIndex = mxFormats->addNew( rFormat, aLocale );
}
mxRangeProps->setPropertyValue( "NumberFormat", uno::makeAny( nNewIndex ) );
- return true;
}
void setNumberFormat( sal_Int16 nType )
diff --git a/slideshow/source/engine/shapes/viewshape.cxx b/slideshow/source/engine/shapes/viewshape.cxx
index e26865accbe8..b10dcda35924 100644
--- a/slideshow/source/engine/shapes/viewshape.cxx
+++ b/slideshow/source/engine/shapes/viewshape.cxx
@@ -817,12 +817,10 @@ namespace slideshow
nYBorder );
}
- bool ViewShape::enterAnimationMode()
+ void ViewShape::enterAnimationMode()
{
mbForceUpdate = true;
mbAnimationMode = true;
-
- return true;
}
void ViewShape::leaveAnimationMode()
diff --git a/slideshow/source/engine/shapes/viewshape.hxx b/slideshow/source/engine/shapes/viewshape.hxx
index 3617dd39b756..b3fc29d27a2d 100644
--- a/slideshow/source/engine/shapes/viewshape.hxx
+++ b/slideshow/source/engine/shapes/viewshape.hxx
@@ -99,10 +99,8 @@ namespace slideshow
This method enters animation mode on the associate
target view. The shape can be animated in parallel on
different views.
-
- @return whether the mode change finished successfully.
*/
- bool enterAnimationMode();
+ void enterAnimationMode();
/** Notify the ViewShape that it is no longer animated
diff --git a/sw/inc/editsh.hxx b/sw/inc/editsh.hxx
index 36c33afa838b..3da4e3617ca0 100644
--- a/sw/inc/editsh.hxx
+++ b/sw/inc/editsh.hxx
@@ -621,10 +621,8 @@ public:
/// Apply ViewOptions with Start-/EndAction.
virtual void ApplyViewOptions( const SwViewOption &rOpt ) override;
- /** Query text within selection.
- @returns FALSE, if selected range is too large to be copied
- into string buffer or if other errors occur. */
- bool GetSelectedText( OUString &rBuf,
+ /** Query text within selection. */
+ void GetSelectedText( OUString &rBuf,
ParaBreakType nHndlParaBreak = ParaBreakType::ToBlank );
/** @return graphic, if CurrentCursor->Point() points to a SwGrfNode
diff --git a/sw/source/core/edit/edglss.cxx b/sw/source/core/edit/edglss.cxx
index 3ece061f823c..2540672a9be3 100644
--- a/sw/source/core/edit/edglss.cxx
+++ b/sw/source/core/edit/edglss.cxx
@@ -117,7 +117,8 @@ sal_uInt16 SwEditShell::SaveGlossaryDoc( SwTextBlocks& rBlock,
pCursor->GetPoint()->nContent.Assign( pContentNd, pContentNd->Len() );
OUString sBuf;
- if( GetSelectedText( sBuf, ParaBreakType::ToOnlyCR ) && !sBuf.isEmpty() )
+ GetSelectedText( sBuf, ParaBreakType::ToOnlyCR );
+ if( !sBuf.isEmpty() )
nRet = rBlock.PutText( rShortName, rName, sBuf );
}
else
@@ -248,10 +249,8 @@ bool SwEditShell::CopySelToDoc( SwDoc* pInsDoc )
}
/** Get text in a Selection
- *
- * @return false if the selected area is too big for being copied into the string buffer
*/
-bool SwEditShell::GetSelectedText( OUString &rBuf, ParaBreakType nHndlParaBrk )
+void SwEditShell::GetSelectedText( OUString &rBuf, ParaBreakType nHndlParaBrk )
{
GetCursor(); // creates all cursors if needed
if( IsSelOnePara() )
@@ -328,8 +327,6 @@ bool SwEditShell::GetSelectedText( OUString &rBuf, ParaBreakType nHndlParaBrk )
}
}
}
-
- return true;
}
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sw/source/uibase/dochdl/gloshdl.cxx b/sw/source/uibase/dochdl/gloshdl.cxx
index 115f3e8d959c..a52a59f38f30 100644
--- a/sw/source/uibase/dochdl/gloshdl.cxx
+++ b/sw/source/uibase/dochdl/gloshdl.cxx
@@ -306,12 +306,7 @@ bool SwGlossaryHdl::NewGlossary(const OUString& rName, const OUString& rShortNam
OUString* pOnlyText = nullptr;
if( bNoAttr )
{
- if( !pWrtShell->GetSelectedText( sOnlyText, ParaBreakType::ToOnlyCR ))
- {
- if (!pCurGrp)
- delete pTmp;
- return false;
- }
+ pWrtShell->GetSelectedText( sOnlyText, ParaBreakType::ToOnlyCR );
pOnlyText = &sOnlyText;
}
More information about the Libreoffice-commits
mailing list