loplugin:unused-returns in sw

Change-Id: I753fcdd35d461880a8d8160213cabd465cfde967
Reviewed-on: https://gerrit.libreoffice.org/48189
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

26 files changed, 75 insertions, 89 deletions

diff --git a/sw/inc/EnhancedPDFExportHelper.hxx b/sw/inc/EnhancedPDFExportHelper.hxx
index 88d5f8c0a393..a0c85e8c85e9 100644
--- a/sw/inc/EnhancedPDFExportHelper.hxx
+++ b/sw/inc/EnhancedPDFExportHelper.hxx
@@ -157,7 +157,7 @@ class SwTaggedPDFHelper
     void EndStructureElements();
 
     bool CheckReopenTag();
-    bool CheckRestoreTag() const;
+    void CheckRestoreTag() const;
 
     public:
 
diff --git a/sw/inc/IDocumentContentOperations.hxx b/sw/inc/IDocumentContentOperations.hxx
index 9f4674ff2953..5ba9b1a6d3fc 100644
--- a/sw/inc/IDocumentContentOperations.hxx
+++ b/sw/inc/IDocumentContentOperations.hxx
@@ -115,7 +115,7 @@ public:
 
     /** Delete a range SwFlyFrameFormat.
     */
-    virtual bool DeleteRange(SwPaM&) = 0;
+    virtual void DeleteRange(SwPaM&) = 0;
 
     /** Delete full paragraphs.
     */
@@ -214,7 +214,7 @@ public:
     virtual bool InsertPoolItem(const SwPaM &rRg, const SfxPoolItem&,
                                 const SetAttrMode nFlags = SetAttrMode::DEFAULT, bool bExpandCharToPara=false) = 0;
 
-    virtual bool InsertItemSet (const SwPaM &rRg, const SfxItemSet&,
+    virtual void InsertItemSet (const SwPaM &rRg, const SfxItemSet&,
         const SetAttrMode nFlags = SetAttrMode::DEFAULT) = 0;
 
     /** Removes any leading white space from the paragraph
diff --git a/sw/inc/IDocumentFieldsAccess.hxx b/sw/inc/IDocumentFieldsAccess.hxx
index c78e5a249b3b..78d0e190b4a2 100644
--- a/sw/inc/IDocumentFieldsAccess.hxx
+++ b/sw/inc/IDocumentFieldsAccess.hxx
@@ -73,7 +73,7 @@ namespace com { namespace sun { namespace star { namespace uno { class Any; } }
        @retval true            putting of value was successful
        @retval false           else
     */
-    virtual bool PutValueToField(const SwPosition & rPos, const css::uno::Any& rVal, sal_uInt16 nWhich) = 0;
+    virtual void PutValueToField(const SwPosition & rPos, const css::uno::Any& rVal, sal_uInt16 nWhich) = 0;
 
     // Call update of expression fields. All expressions are re-evaluated.
 
diff --git a/sw/inc/authfld.hxx b/sw/inc/authfld.hxx
index e221dd031bbc..d861ddd0fd8b 100644
--- a/sw/inc/authfld.hxx
+++ b/sw/inc/authfld.hxx
@@ -80,14 +80,14 @@ public:
 
     virtual SwFieldType* Copy()    const override;
 
-    virtual bool        QueryValue( css::uno::Any& rVal, sal_uInt16 nWhichId ) const override;
-    virtual bool        PutValue( const css::uno::Any& rVal, sal_uInt16 nWhichId ) override;
+    virtual void        QueryValue( css::uno::Any& rVal, sal_uInt16 nWhichId ) const override;
+    virtual void        PutValue( const css::uno::Any& rVal, sal_uInt16 nWhichId ) override;
 
     void         SetDoc(SwDoc* pNewDoc)              { m_pDoc = pNewDoc; }
     SwDoc*              GetDoc(){ return m_pDoc; }
     void                RemoveField(sal_IntPtr nHandle);
     sal_IntPtr          AddField(const OUString& rFieldContents);
-    bool            AddField(sal_IntPtr nHandle);
+    void                AddField(sal_IntPtr nHandle);
     void                DelSequenceArray()
                         {
                             m_SequArr.clear();
diff --git a/sw/inc/crsrsh.hxx b/sw/inc/crsrsh.hxx
index fd75609f057f..515ee873e9e2 100644
--- a/sw/inc/crsrsh.hxx
+++ b/sw/inc/crsrsh.hxx
@@ -303,7 +303,7 @@ public:
 
     SwPaM * CreateCursor();
     ///< delete the current cursor and make the following into the current
-    bool DestroyCursor();
+    void DestroyCursor();
     ///< transform TableCursor to normal cursor, nullify Tablemode
     void TableCursorToCursor();
     ///< enter block mode, change normal cursor into block cursor
@@ -358,7 +358,7 @@ public:
     bool MovePara( SwWhichPara, SwMoveFnCollection const & );
     bool MoveSection( SwWhichSection, SwMoveFnCollection const & );
     bool MoveTable( SwWhichTable, SwMoveFnCollection const & );
-    bool MoveColumn( SwWhichColumn, SwPosColumn );
+    void MoveColumn( SwWhichColumn, SwPosColumn );
     bool MoveRegion( SwWhichRegion, SwMoveFnCollection const & );
 
     sal_uLong Find( const i18nutil::SearchOptions2& rSearchOpt,
@@ -518,7 +518,7 @@ public:
     /*
      * Returns SwRect, at which the character is located.
      */
-    bool GetCharRectAt(SwRect& rRect, const SwPosition* pPos);
+    void GetCharRectAt(SwRect& rRect, const SwPosition* pPos);
 
     // Return current page number:
     // true:  in which cursor is located.
@@ -593,8 +593,8 @@ public:
 
     bool SelTable();
 
-    bool GotoNextNum();
-    bool GotoPrevNum();
+    void GotoNextNum();
+    void GotoPrevNum();
 
     bool GotoOutline( const OUString& rName );
     // to the next/previous or the given OutlineNode
@@ -641,7 +641,7 @@ public:
     bool GotoPrevFootnoteAnchor();
     bool GotoNextFootnoteAnchor();
 
-    bool GotoFlyAnchor();       ///< jump from the frame to the anchor
+    void GotoFlyAnchor();       ///< jump from the frame to the anchor
     bool GotoHeaderText();       ///< jump from the content to the header
     bool GotoFooterText();       ///< jump from the content to the footer
     // jump to the header/footer of the given or current PageDesc
@@ -652,7 +652,7 @@ public:
 
     bool GotoNextTOXBase( const OUString* = nullptr );
     bool GotoPrevTOXBase( const OUString* = nullptr );
-    bool GotoTOXMarkBase();
+    void GotoTOXMarkBase();
     // jump to the next or previous index entry
     bool GotoNxtPrvTOXMark( bool bNext = true );
     // jump to the next/previous index mark of this type
@@ -714,7 +714,7 @@ public:
     bool GoStartSentence();
     bool GoEndSentence();
     bool SelectWord( const Point* pPt );
-    bool ExpandToSentenceBorders();
+    void ExpandToSentenceBorders();
 
     // get position from current cursor
     bool IsStartWord( sal_Int16 nWordType = css::i18n::WordType::ANYWORD_IGNOREWHITESPACES )const;
diff --git a/sw/inc/dbfld.hxx b/sw/inc/dbfld.hxx
index cb7ed9c266bf..214d3d9acec3 100644
--- a/sw/inc/dbfld.hxx
+++ b/sw/inc/dbfld.hxx
@@ -47,8 +47,8 @@ public:
     const OUString&     GetColumnName() const {return sColumn;}
     const SwDBData& GetDBData() const {return aDBData;}
 
-    virtual bool        QueryValue( css::uno::Any& rVal, sal_uInt16 nWhich ) const override;
-    virtual bool        PutValue( const css::uno::Any& rVal, sal_uInt16 nWhich ) override;
+    virtual void        QueryValue( css::uno::Any& rVal, sal_uInt16 nWhich ) const override;
+    virtual void        PutValue( const css::uno::Any& rVal, sal_uInt16 nWhich ) override;
 };
 
 // Classes derived from SwFields. They overlay the expand-function.
diff --git a/sw/inc/dbmgr.hxx b/sw/inc/dbmgr.hxx
index 555d4dc9079f..b4a6f9e8abfe 100644
--- a/sw/inc/dbmgr.hxx
+++ b/sw/inc/dbmgr.hxx
@@ -372,7 +372,7 @@ public:
                                       OUString &rResult, double *pNumber);
     bool            FillCalcWithMergeData(SvNumberFormatter *pDocFormatter,
                                           LanguageType nLanguage, SwCalc &aCalc);
-    bool            ToNextRecord(const OUString& rDataSource, const OUString& rTableOrQuery);
+    void            ToNextRecord(const OUString& rDataSource, const OUString& rTableOrQuery);
 
     sal_uInt32      GetSelectedRecordId();
     bool            ToRecordId(sal_Int32 nSet);
diff --git a/sw/inc/ddefld.hxx b/sw/inc/ddefld.hxx
index a583f247a966..e8c444676e16 100644
--- a/sw/inc/ddefld.hxx
+++ b/sw/inc/ddefld.hxx
@@ -52,8 +52,8 @@ public:
     virtual SwFieldType* Copy() const override;
     virtual OUString GetName() const override;
 
-    virtual bool QueryValue( css::uno::Any& rVal, sal_uInt16 nWhich ) const override;
-    virtual bool PutValue( const css::uno::Any& rVal, sal_uInt16 nWhich ) override;
+    virtual void QueryValue( css::uno::Any& rVal, sal_uInt16 nWhich ) const override;
+    virtual void PutValue( const css::uno::Any& rVal, sal_uInt16 nWhich ) override;
 
     OUString const & GetCmd() const;
     void SetCmd( const OUString& aStr );
diff --git a/sw/inc/doc.hxx b/sw/inc/doc.hxx
index 49cc1bd41551..903cd5b910de 100644
--- a/sw/inc/doc.hxx
+++ b/sw/inc/doc.hxx
@@ -513,7 +513,7 @@ public:
     ::sw::DocumentContentOperationsManager & GetDocumentContentOperationsManager();
 
     bool UpdateParRsid( SwTextNode *pTextNode, sal_uInt32 nVal = 0 );
-    bool UpdateRsid( const SwPaM &rRg, sal_Int32 nLen );
+    void UpdateRsid( const SwPaM &rRg, sal_Int32 nLen );
 
     // IDocumentStylePoolAccess
     IDocumentStylePoolAccess const & getIDocumentStylePoolAccess() const;
@@ -1183,19 +1183,19 @@ public:
     bool TableToText( const SwTableNode* pTableNd, sal_Unicode cCh );
 
     // Create columns / rows in table.
-    bool InsertCol( const SwCursor& rCursor,
+    void InsertCol( const SwCursor& rCursor,
                     sal_uInt16 nCnt = 1, bool bBehind = true );
     bool InsertCol( const SwSelBoxes& rBoxes,
                     sal_uInt16 nCnt = 1, bool bBehind = true );
-    bool InsertRow( const SwCursor& rCursor,
+    void InsertRow( const SwCursor& rCursor,
                     sal_uInt16 nCnt = 1, bool bBehind = true );
     bool InsertRow( const SwSelBoxes& rBoxes,
                     sal_uInt16 nCnt = 1, bool bBehind = true );
 
     // Delete Columns/Rows in table.
     bool DeleteRowCol( const SwSelBoxes& rBoxes, bool bColumn = false );
-    bool DeleteRow( const SwCursor& rCursor );
-    bool DeleteCol( const SwCursor& rCursor );
+    void DeleteRow( const SwCursor& rCursor );
+    void DeleteCol( const SwCursor& rCursor );
 
     // Split / concatenate boxes in table.
     bool SplitTable( const SwSelBoxes& rBoxes, bool bVert,
@@ -1265,9 +1265,9 @@ public:
                         const SwTable* pCpyTable, bool bCpyName = false,
                         bool bCorrPos = false );
 
-    bool UnProtectCells( const OUString& rTableName );
+    void UnProtectCells( const OUString& rTableName );
     bool UnProtectCells( const SwSelBoxes& rBoxes );
-    bool UnProtectTables( const SwPaM& rPam );
+    void UnProtectTables( const SwPaM& rPam );
     bool HasTableAnyProtection( const SwPosition* pPos,
                               const OUString* pTableName,
                               bool* pFullTableProtection );
@@ -1390,7 +1390,7 @@ public:
     const SwFormatINetFormat* FindINetAttr( const OUString& rName ) const;
 
     // Call into intransparent Basic; expect possible Return String.
-    bool ExecMacro( const SvxMacro& rMacro, OUString* pRet, SbxArray* pArgs );
+    void ExecMacro( const SvxMacro& rMacro, OUString* pRet, SbxArray* pArgs );
 
     // Call into intransparent Basic / JavaScript.
     sal_uInt16 CallEvent( SvMacroItemId nEvent, const SwCallMouseEvent& rCallEvent,
diff --git a/sw/inc/docary.hxx b/sw/inc/docary.hxx
index 7f0aeb772f69..d616a0b839a7 100644
--- a/sw/inc/docary.hxx
+++ b/sw/inc/docary.hxx
@@ -340,7 +340,7 @@ public:
     bool InsertWithValidRanges(SwRangeRedlinePtr& p, size_type* pInsPos = nullptr);
 
     void Remove( size_type nPos );
-    bool Remove( const SwRangeRedline* p );
+    void Remove( const SwRangeRedline* p );
     void DeleteAndDestroy( size_type nPos, size_type nLen = 1 );
     void DeleteAndDestroyAll();
 
@@ -383,7 +383,7 @@ private:
 public:
     ~SwExtraRedlineTable();
 
-    bool Insert( SwExtraRedline* p );
+    void Insert( SwExtraRedline* p );
 
     void DeleteAndDestroy( sal_uInt16 nPos, sal_uInt16 nLen = 1 );
     void DeleteAndDestroyAll();
diff --git a/sw/inc/editsh.hxx b/sw/inc/editsh.hxx
index 197d3b274525..d290fd814130 100644
--- a/sw/inc/editsh.hxx
+++ b/sw/inc/editsh.hxx
@@ -222,7 +222,7 @@ public:
 
     /** If cursor is in a INetAttribute it will be deleted completely
      including the descriptive text (needed at drag & drop). */
-    bool DelINetAttrWithText();
+    void DelINetAttrWithText();
 
     /** If Cursor is at the end of a character style in which the DontExpand-flag
      is not yet set, the latter will be set (==> return TRUE). */
@@ -258,11 +258,8 @@ public:
      * @param rSet
      * output parameter - the SfxItemSet where the automatic paragraph format attribute(s) will be store.
      * The attributes aren't invalidated or cleared if the function reach the getMaxLookup limit.
-     *
-     * @return true if the function inspect all the nodes point by the pPaM parameter,
-     * false if the function reach the limit of getMaxLookup number of nodes inspected.
      */
-    bool GetCurParAttr( SfxItemSet& rSet ) const;
+    void GetCurParAttr( SfxItemSet& rSet ) const;
     /**
      * Get the paragraph format attribute(s) of the selection(s) described by a SwPaM.
      *
@@ -523,12 +520,12 @@ public:
                         const bool bResetIndentAttrs = false );
 
     /// Paragraphs without enumeration but with indents.
-    bool NoNum();
+    void NoNum();
 
     /// Delete, split enumeration list.
     void DelNumRules();
 
-    bool NumUpDown( bool bDown = true );
+    void NumUpDown( bool bDown = true );
 
     bool MoveParagraph( long nOffset = 1);
     bool MoveNumParas( bool bUpperLower, bool bUpperLeft );
@@ -575,7 +572,7 @@ public:
 
     sal_uInt16 GetNodeNumStart( SwPaM* pPaM ) const;
 
-    bool ReplaceNumRule( const OUString& rOldRule, const OUString& rNewRule );
+    void ReplaceNumRule( const OUString& rOldRule, const OUString& rNewRule );
 
     /** Searches for a text node with a numbering rule.
      in case a list style is found, <sListId> holds the list id, to which the
@@ -612,9 +609,9 @@ public:
     /// should only be called by sw::UndoManager!
     void HandleUndoRedoContext(::sw::UndoRedoContext & rContext);
 
-    bool Undo(sal_uInt16 const nCount = 1);
-    bool Redo(sal_uInt16 const nCount = 1);
-    bool Repeat(sal_uInt16 const nCount);
+    void Undo(sal_uInt16 const nCount = 1);
+    void Redo(sal_uInt16 const nCount = 1);
+    void Repeat(sal_uInt16 const nCount);
 
     /// For all views of this document.
     void StartAllAction();
@@ -750,7 +747,7 @@ public:
     void SetTableChgMode( TableChgMode eMode );
 
     /// Split table at cursor position.
-    bool SplitTable( SplitTable_HeadlineOption eMode );
+    void SplitTable( SplitTable_HeadlineOption eMode );
 
     /** Merge tables.
 
@@ -886,7 +883,7 @@ public:
      section/table. The purpose of the method is to allow users to inert text
      at certain 'impossible' position, e.g. before a table at the document
      start or between to sections. */
-    bool DoSpecialInsert();
+    void DoSpecialInsert();
     bool CanSpecialInsert() const;
 
     /// Optimizing UI.
@@ -911,17 +908,17 @@ public:
     void SetGlblDocSaveLinks( bool bFlag );
     bool IsGlblDocSaveLinks() const;
     void GetGlobalDocContent( SwGlblDocContents& rArr ) const;
-    bool InsertGlobalDocContent( const SwGlblDocContent& rPos,
+    void InsertGlobalDocContent( const SwGlblDocContent& rPos,
                                  SwSectionData & rNew );
     bool InsertGlobalDocContent( const SwGlblDocContent& rPos,
                                  const SwTOXBase& rTOX );
     bool InsertGlobalDocContent( const SwGlblDocContent& rPos );
-    bool DeleteGlobalDocContent( const SwGlblDocContents& rArr,
+    void DeleteGlobalDocContent( const SwGlblDocContents& rArr,
                                 size_t nPos );
     bool MoveGlobalDocContent( const SwGlblDocContents& rArr ,
                                 size_t nFromPos, size_t nToPos,
                                 size_t nNewPos );
-    bool GotoGlobalDocContent( const SwGlblDocContent& rPos );
+    void GotoGlobalDocContent( const SwGlblDocContent& rPos );
 
     /// For Redlining.
     RedlineFlags GetRedlineFlags() const;
diff --git a/sw/inc/expfld.hxx b/sw/inc/expfld.hxx
index 0b55cb8d8ca1..aefbbdbcbe0e 100644
--- a/sw/inc/expfld.hxx
+++ b/sw/inc/expfld.hxx
@@ -188,8 +188,8 @@ public:
     const SwNode* GetOutlineChgNd() const   { return pOutlChgNd; }
     void SetOutlineChgNd( const SwNode* p ) { pOutlChgNd = p; }
 
-    virtual bool        QueryValue( css::uno::Any& rVal, sal_uInt16 nWhich ) const override;
-    virtual bool        PutValue( const css::uno::Any& rVal, sal_uInt16 nWhich ) override;
+    virtual void        QueryValue( css::uno::Any& rVal, sal_uInt16 nWhich ) const override;
+    virtual void        PutValue( const css::uno::Any& rVal, sal_uInt16 nWhich ) override;
 };
 
 inline void SwSetExpFieldType::SetType( sal_uInt16 nTyp )
diff --git a/sw/inc/fesh.hxx b/sw/inc/fesh.hxx
index 3e5b719d5a9f..9923aad5bff6 100644
--- a/sw/inc/fesh.hxx
+++ b/sw/inc/fesh.hxx
@@ -255,11 +255,11 @@ public:
     virtual ~SwFEShell() override;
 
     /// Copy and Paste methods for internal clipboard.
-    bool Copy( SwDoc* pClpDoc, const OUString* pNewClpText = nullptr );
+    void Copy( SwDoc* pClpDoc, const OUString* pNewClpText = nullptr );
     bool Paste( SwDoc* pClpDoc );
 
     /// Paste some pages into another doc - used in mailmerge.
-    bool PastePages( SwFEShell& rToFill, sal_uInt16 nStartPage, sal_uInt16 nEndPage);
+    void PastePages( SwFEShell& rToFill, sal_uInt16 nStartPage, sal_uInt16 nEndPage);
 
     /// Copy-Method for Drag&Drop
     bool Copy( SwFEShell*, const Point& rSttPt, const Point& rInsPt,
@@ -382,7 +382,7 @@ public:
     bool GetFlyFrameAttr( SfxItemSet &rSet ) const;
     bool SetFlyFrameAttr( SfxItemSet &rSet );
     static SfxItemSet makeItemSetFromFormatAnchor(SfxItemPool& rPool, const SwFormatAnchor &rAnchor);
-    bool ResetFlyFrameAttr( const SfxItemSet* pSet );
+    void ResetFlyFrameAttr( const SfxItemSet* pSet );
     const SwFrameFormat *NewFlyFrame( const SfxItemSet &rSet, bool bAnchValid = false,
                          SwFrameFormat *pParent = nullptr );
     void SetFlyPos( const Point &rAbsPos);
@@ -494,7 +494,7 @@ public:
 
     /// Attention: Ambiguities if multiple selections.
     bool GetObjAttr( SfxItemSet &rSet ) const;
-    bool SetObjAttr( const SfxItemSet &rSet );
+    void SetObjAttr( const SfxItemSet &rSet );
 
     const SdrObject* GetBestObject( bool bNext, GotoObjFlags eType, bool bFlat = true, const svx::ISdrObjectFilter* pFilter = nullptr );
     bool GotoObj( bool bNext, GotoObjFlags eType = GotoObjFlags::DrawAny);
@@ -579,7 +579,7 @@ public:
     void InsertDrawObj( SdrObject& rDrawObj,
                         const Point& rInsertPosition );
 
-    bool ReplaceSdrObj( const OUString& rGrfName, const Graphic* pGrf );
+    void ReplaceSdrObj( const OUString& rGrfName, const Graphic* pGrf );
 
     // --> #i972#
     /** for starmath formulas anchored 'as char' it aligns it baseline to baseline
@@ -635,10 +635,10 @@ public:
     /// Is content of a table cell or at least a table cell completely selected?
     bool HasBoxSelection() const;
 
-    bool InsertRow( sal_uInt16 nCnt, bool bBehind );
-    bool InsertCol( sal_uInt16 nCnt, bool bBehind );  // 0 == at the end.
+    void InsertRow( sal_uInt16 nCnt, bool bBehind );
+    void InsertCol( sal_uInt16 nCnt, bool bBehind );  // 0 == at the end.
     bool DeleteCol();
-    bool DeleteTable();
+    void DeleteTable();
     bool DeleteRow(bool bCompleteTable = false);
 
     bool DeleteTableSel();        ///< Current selection, may be whole table.
@@ -646,7 +646,7 @@ public:
     TableMergeErr MergeTab();          /**< Merge selected parts of table */
 
     /// Split cell vertically or horizontally.
-    bool SplitTab( bool bVert, sal_uInt16 nCnt, bool bSameHeight );
+    void SplitTab( bool bVert, sal_uInt16 nCnt, bool bSameHeight );
     bool Sort(const SwSortOptions&);    //Sortieren.
 
     void SetRowHeight( const SwFormatFrameSize &rSz );
@@ -733,9 +733,9 @@ public:
 
     bool GetTableAutoFormat( SwTableAutoFormat& rGet );
 
-    bool SetColRowWidthHeight( TableChgWidthHeightType eType, sal_uInt16 nDiff );
+    void SetColRowWidthHeight( TableChgWidthHeightType eType, sal_uInt16 nDiff );
 
-    bool GetAutoSum( OUString& rFormula ) const;
+    void GetAutoSum( OUString& rFormula ) const;
 
     /** Phy: real page count.
      Virt: consider offset that may have been set by user. */
diff --git a/sw/inc/fldbas.hxx b/sw/inc/fldbas.hxx
index a71fdb715a15..e3fed4461a70 100644
--- a/sw/inc/fldbas.hxx
+++ b/sw/inc/fldbas.hxx
@@ -257,8 +257,8 @@ public:
     /// Only in derived classes.
     virtual OUString        GetName() const;
     virtual SwFieldType*    Copy()    const = 0;
-    virtual bool QueryValue( css::uno::Any& rVal, sal_uInt16 nWhich ) const;
-    virtual bool PutValue( const css::uno::Any& rVal, sal_uInt16 nWhich );
+    virtual void QueryValue( css::uno::Any& rVal, sal_uInt16 nWhich ) const;
+    virtual void PutValue( const css::uno::Any& rVal, sal_uInt16 nWhich );
 
     SwFieldIds              Which() const { return m_nWhich; }
 
diff --git a/sw/inc/flddropdown.hxx b/sw/inc/flddropdown.hxx
index e7bdfbbd4375..81074d8fabca 100644
--- a/sw/inc/flddropdown.hxx
+++ b/sw/inc/flddropdown.hxx
@@ -211,11 +211,8 @@ public:
        rItem is not found the selection will be empty.
 
        @param rItem the item to be set
-
-       @retval true the selected item was successfully set
-       @retval false failure (empty selection)
     */
-    bool SetSelectedItem(const OUString & rItem);
+    void SetSelectedItem(const OUString & rItem);
 
     /**
        Sets the name of the field.
diff --git a/sw/inc/fmtcol.hxx b/sw/inc/fmtcol.hxx
index 80172c827de4..48cbedb3d99e 100644
--- a/sw/inc/fmtcol.hxx
+++ b/sw/inc/fmtcol.hxx
@@ -217,7 +217,7 @@ public:
     const SwCollCondition* HasCondition( const SwCollCondition& rCond ) const;
     const SwFormatCollConditions& GetCondColls() const { return m_CondColls; }
     void InsertCondition( const SwCollCondition& rCond );
-    bool RemoveCondition( const SwCollCondition& rCond );
+    void RemoveCondition( const SwCollCondition& rCond );
 
     void SetConditions( const SwFormatCollConditions& );
 };
diff --git a/sw/inc/hhcwrp.hxx b/sw/inc/hhcwrp.hxx
index 1401abdc10a9..d77b291e85b0 100644
--- a/sw/inc/hhcwrp.hxx
+++ b/sw/inc/hhcwrp.hxx
@@ -53,7 +53,7 @@ class SW_DLLPUBLIC SwHHCWrapper : public editeng::HangulHanjaConversion
 
     /// from SvxSpellWrapper copied and modified
     bool        ConvNext_impl();        ///< former SpellNext
-    bool        FindConvText_impl();    ///< former FindSpellError
+    void        FindConvText_impl();    ///< former FindSpellError
 
     void        ConvStart_impl( SwConversionArgs *pConvArgs, SvxSpellArea eSpell );   ///< former SpellStart
     void        ConvEnd_impl( SwConversionArgs const *pConvArgs );                          ///< former SpellEnd
diff --git a/sw/inc/ndtxt.hxx b/sw/inc/ndtxt.hxx
index 78e9921ce6e0..941115235e56 100644
--- a/sw/inc/ndtxt.hxx
+++ b/sw/inc/ndtxt.hxx
@@ -193,7 +193,7 @@ public:
     SwGrammarMarkUp* GetGrammarCheck();
     void SetSmartTags( SwWrongList* pNew, bool bDelete = true );
     SwWrongList* GetSmartTags();
-    bool TryCharSetExpandToNum(const SfxItemSet& pCharSet);
+    void TryCharSetExpandToNum(const SfxItemSet& pCharSet);
 
     /// End: Data collected during idle time
 
diff --git a/sw/inc/pagepreviewlayout.hxx b/sw/inc/pagepreviewlayout.hxx
index 810ca1e8cc22..cf5227a1c1bf 100644
--- a/sw/inc/pagepreviewlayout.hxx
+++ b/sw/inc/pagepreviewlayout.hxx
@@ -180,10 +180,8 @@ private:
 
         @param _opPreviewPage
         output parameter - calculated preview data.
-
-        @return boolean, indicating, if calculation was successful.
     */
-    bool CalcPreviewDataForPage( const SwPageFrame& _rPage,
+    void CalcPreviewDataForPage( const SwPageFrame& _rPage,
                                   const Point& _rPreviewOffset,
                                   PreviewPage* _opPreviewPage );
 
@@ -266,10 +264,8 @@ public:
         @param _rPxWinSize
         input parameter - window size in which the preview will be displayed and
         for which the scaling will be calculated.
-
-        @return boolean, indicating, if preview layout is successful initialized.
     */
-    bool Init( const sal_uInt16 _nCols,
+    void Init( const sal_uInt16 _nCols,
                const sal_uInt16 _nRows,
                const Size&      _rPxWinSize
               );
@@ -277,10 +273,8 @@ public:
     /** method to adjust page preview layout to document changes
 
         @author OD
-
-        @return boolean, indicating, if preview layout is successful initialized.
     */
-    bool ReInit();
+    void ReInit();
 
     /** prepare paint of page preview
 
@@ -427,10 +421,8 @@ public:
 
         @param _orNewStartPos
         output parameter - new start position in document preview
-
-        @return boolean - indicating, that move was successful.
     */
-    bool CalcStartValuesForSelectedPageMove( const sal_Int16  _nHoriMove,
+    void CalcStartValuesForSelectedPageMove( const sal_Int16  _nHoriMove,
                                              const sal_Int16  _nVertMove,
                                              sal_uInt16&      _orNewSelectedPage,
                                              sal_uInt16&      _orNewStartPage,
diff --git a/sw/inc/swabstdlg.hxx b/sw/inc/swabstdlg.hxx
index 61e64265dfe3..615b40552fb8 100644
--- a/sw/inc/swabstdlg.hxx
+++ b/sw/inc/swabstdlg.hxx
@@ -371,7 +371,7 @@ protected:
     virtual ~AbstractMailMergeWizard() override = default;
 public:
     virtual OUString            GetReloadDocument() const = 0;
-    virtual bool            ShowPage( sal_uInt16 nLevel ) = 0;
+    virtual void                ShowPage( sal_uInt16 nLevel ) = 0;
     virtual sal_uInt16          GetRestartPage() const = 0;
 };
 
diff --git a/sw/inc/swbaslnk.hxx b/sw/inc/swbaslnk.hxx
index 539f430ddee2..4f922e698f95 100644
--- a/sw/inc/swbaslnk.hxx
+++ b/sw/inc/swbaslnk.hxx
@@ -49,7 +49,7 @@ public:
     // For graphics only.
     bool SwapIn( bool bWaitForData = false, bool bNativFormat = false );
 
-    bool Connect() { return nullptr != SvBaseLink::GetRealObject(); }
+    void Connect() { SvBaseLink::GetRealObject(); }
 
     // Only for graphics-links (for switching between DDE / Grf-link).
     using SvBaseLink::SetObjType;
diff --git a/sw/inc/swtable.hxx b/sw/inc/swtable.hxx
index 1d240de469ba..b41e9990f502 100644
--- a/sw/inc/swtable.hxx
+++ b/sw/inc/swtable.hxx
@@ -289,7 +289,7 @@ public:
     bool InsNewTable( const SwTable& rCpyTable, const SwSelBoxes&,
                       SwUndoTableCpyTable* pUndo );
     // Copy headline of table (with content!) into an other one.
-    bool CopyHeadlineIntoTable( SwTableNode& rTableNd );
+    void CopyHeadlineIntoTable( SwTableNode& rTableNd );
 
     // Get box, whose start index is set on nBoxStt.
           SwTableBox* GetTableBox( sal_uLong nSttIdx );
diff --git a/sw/inc/tblafmt.hxx b/sw/inc/tblafmt.hxx
index 797989fa58b9..11ea7488cd73 100644
--- a/sw/inc/tblafmt.hxx
+++ b/sw/inc/tblafmt.hxx
@@ -188,7 +188,7 @@ public:
 
     bool Load( SvStream& rStream, const SwAfVersions& rVersions, sal_uInt16 nVer );
     bool Save( SvStream& rStream, sal_uInt16 fileVersion ) const;
-    bool SaveVersionNo( SvStream& rStream, sal_uInt16 fileVersion ) const;
+    void SaveVersionNo( SvStream& rStream, sal_uInt16 fileVersion ) const;
 };
 
 /*
@@ -366,7 +366,7 @@ public:
     /// Find table style with the provided name, return nullptr when not found.
     SwTableAutoFormat* FindAutoFormat(const OUString& rName) const;
 
-    bool Load();
+    void Load();
     bool Save() const;
 };
 
diff --git a/sw/inc/txatritr.hxx b/sw/inc/txatritr.hxx
index 6827d73191c8..96e4c296c3f3 100644
--- a/sw/inc/txatritr.hxx
+++ b/sw/inc/txatritr.hxx
@@ -41,7 +41,7 @@ public:
     SwScriptIterator( const OUString& rStr, sal_Int32 nStart,
                       bool bFrwrd = true );
 
-    bool Next();
+    void Next();
 
     sal_uInt16 GetCurrScript() const { return nCurScript; }
     sal_Int32 GetScriptChgPos() const { return (m_nChgPos == -1) ? COMPLETE_STRING : m_nChgPos; }
diff --git a/sw/inc/unochart.hxx b/sw/inc/unochart.hxx
index 4ad0b0caaf8c..17708e6f69f7 100644
--- a/sw/inc/unochart.hxx
+++ b/sw/inc/unochart.hxx
@@ -321,7 +321,7 @@ public:
     bool    DeleteBox( const SwTableBox &rBox );
 
     void        FillRangeDesc( SwRangeDescriptor &rRangeDesc ) const;
-    bool        ExtendTo( bool bExtendCol, sal_Int32 nFirstNew, sal_Int32 nCount );
+    void        ExtendTo( bool bExtendCol, sal_Int32 nFirstNew, sal_Int32 nCount );
     std::vector< css::uno::Reference< css::table::XCell > > GetCells();
 };
 
diff --git a/sw/inc/usrfld.hxx b/sw/inc/usrfld.hxx
index 63ebabc2f220..d523964b97c3 100644
--- a/sw/inc/usrfld.hxx
+++ b/sw/inc/usrfld.hxx
@@ -58,8 +58,8 @@ public:
     bool                    IsDeleted() const       { return bDeleted; }
     void                    SetDeleted( bool b )    { bDeleted = b; }
 
-    virtual bool        QueryValue( css::uno::Any& rVal, sal_uInt16 nMId ) const override;
-    virtual bool        PutValue( const css::uno::Any& rVal, sal_uInt16 nMId ) override;
+    virtual void        QueryValue( css::uno::Any& rVal, sal_uInt16 nMId ) const override;
+    virtual void        PutValue( const css::uno::Any& rVal, sal_uInt16 nMId ) override;
 
 protected:
     virtual void Modify( const SfxPoolItem* pOld, const SfxPoolItem* pNew ) override;