loplugin:unusedmethods unused return value in include/svtools

Change-Id: Ibc83f032bb703559cc643933e53e8670bd6560b3

18 files changed, 25 insertions, 35 deletions

diff --git a/include/svtools/accessibletable.hxx b/include/svtools/accessibletable.hxx
index d5d838f309d7..3615c82550c7 100644
--- a/include/svtools/accessibletable.hxx
+++ b/include/svtools/accessibletable.hxx
@@ -64,7 +64,7 @@ public:
     virtual sal_Int32               GetCurrentColumn() const = 0;
     virtual css::uno::Reference< css::accessibility::XAccessible > CreateAccessibleControl( sal_Int32 _nIndex )= 0;
     virtual OUString GetAccessibleObjectName(AccessibleTableControlObjType eObjType, sal_Int32 _nRow, sal_Int32 _nCol) const= 0;
-    virtual bool    GoToCell( sal_Int32 _nColumnPos, sal_Int32 _nRow )= 0;
+    virtual void    GoToCell( sal_Int32 _nColumnPos, sal_Int32 _nRow )= 0;
     virtual bool    HasColHeader() = 0;
     virtual bool    HasRowHeader() = 0;
 
diff --git a/include/svtools/brwbox.hxx b/include/svtools/brwbox.hxx
index 2b641f9b8944..cad8aaaf1d3e 100644
--- a/include/svtools/brwbox.hxx
+++ b/include/svtools/brwbox.hxx
@@ -500,7 +500,7 @@ public:
     // movement of visible area
     long            ScrollColumns( long nColumns );
     long            ScrollRows( long nRows );
-    bool            MakeFieldVisible( long nRow, sal_uInt16 nColId, bool bComplete = false );
+    void            MakeFieldVisible( long nRow, sal_uInt16 nColId, bool bComplete = false );
 
     // access and movement of cursor
     long            GetCurRow() const { return nCurRow; }
diff --git a/include/svtools/contextmenuhelper.hxx b/include/svtools/contextmenuhelper.hxx
index b636fddf9519..f0927967883e 100644
--- a/include/svtools/contextmenuhelper.hxx
+++ b/include/svtools/contextmenuhelper.hxx
@@ -85,7 +85,7 @@ class SVT_DLLPUBLIC ContextMenuHelper
 
         // creates an association between current module/controller bound to the
         // provided frame and their ui configuration managers.
-        bool            associateUIConfigurationManagers();
+        void            associateUIConfigurationManagers();
 
         // resets associations to create associations again on-demand.
         // Useful for implementations which recycle frames. Normal
diff --git a/include/svtools/ctrlbox.hxx b/include/svtools/ctrlbox.hxx
index cf38dc3eae75..f4b5dd5105c6 100644
--- a/include/svtools/ctrlbox.hxx
+++ b/include/svtools/ctrlbox.hxx
@@ -296,7 +296,7 @@ class SVT_DLLPUBLIC LineListBox : public ListBox
                                     sal_uInt16 nStyle, Bitmap& rBmp );
     using Window::ImplInit;
     SVT_DLLPRIVATE void         ImplInit();
-    bool            UpdatePaintLineColor();       // returns sal_True if maPaintCol has changed
+    void            UpdatePaintLineColor();       // returns sal_True if maPaintCol has changed
     virtual void    DataChanged( const DataChangedEvent& rDCEvt ) override;
 
     void            UpdateEntries( long nOldWidth );
diff --git a/include/svtools/editbrowsebox.hxx b/include/svtools/editbrowsebox.hxx
index 00af59b1abd2..7af671faf9ab 100644
--- a/include/svtools/editbrowsebox.hxx
+++ b/include/svtools/editbrowsebox.hxx
@@ -466,7 +466,6 @@ namespace svt
             bool Is() const {return pEvent != nullptr;}
             bool IsDown() const {return bDown;}
             const BrowserMouseEvent* operator->() const {return pEvent;}
-            const BrowserMouseEvent& operator*() const {return *pEvent;}
 
             SVT_DLLPUBLIC void Clear();
             void Set(const BrowserMouseEvent* pEvt, bool bIsDown);
diff --git a/include/svtools/ehdl.hxx b/include/svtools/ehdl.hxx
index 23f5ce7a34ab..a6c2286defaa 100644
--- a/include/svtools/ehdl.hxx
+++ b/include/svtools/ehdl.hxx
@@ -65,7 +65,7 @@ private:
     ResMgr              *pMgr;
     ResMgr              *pFreeMgr;
 
-    SVT_DLLPRIVATE bool   GetClassString(sal_uLong lErrId, OUString &) const;
+    SVT_DLLPRIVATE void   GetClassString(sal_uLong lErrId, OUString &) const;
     virtual bool          CreateString( const ErrorInfo *, OUString &, sal_uInt16 &) const override;
 };
 
diff --git a/include/svtools/extcolorcfg.hxx b/include/svtools/extcolorcfg.hxx
index 1f0db36ef777..735c45189d37 100644
--- a/include/svtools/extcolorcfg.hxx
+++ b/include/svtools/extcolorcfg.hxx
@@ -56,9 +56,6 @@ public:
     inline sal_Int32       getDefaultColor() const { return m_nDefaultColor; }
 
     inline void            setColor(sal_Int32 _nColor) { m_nColor = _nColor; }
-
-    bool operator !=(const ExtendedColorConfigValue& rCmp) const
-        { return m_nColor != rCmp.m_nColor;}
 };
 
 class SVT_DLLPUBLIC ExtendedColorConfig : public SfxBroadcaster, public SfxListener
diff --git a/include/svtools/grfmgr.hxx b/include/svtools/grfmgr.hxx
index 8e50fe3e4aec..7b31fd5860c3 100644
--- a/include/svtools/grfmgr.hxx
+++ b/include/svtools/grfmgr.hxx
@@ -459,10 +459,8 @@ public:
         tile bitmap is smaller. Otherwise, the tile is drawn as
         is. This is useful if e.g. you want only a few, very large
         bitmap drawings appear on the outdev.
-
-        @return sal_True, if drawing completed successfully
      */
-    bool                    DrawTiled(
+    void                    DrawTiled(
                                 OutputDevice* pOut,
                                 const Rectangle& rArea,
                                 const Size& rSize,
diff --git a/include/svtools/imap.hxx b/include/svtools/imap.hxx
index 177545761e55..900e674618b5 100644
--- a/include/svtools/imap.hxx
+++ b/include/svtools/imap.hxx
@@ -130,7 +130,6 @@ class IMapCompat
                     IMapCompat() {}
                     IMapCompat( const IMapCompat& ) {}
     IMapCompat&     operator=( const IMapCompat& ) { return *this; }
-    bool            operator==( const IMapCompat& ) { return false; }
 
 public:
 
diff --git a/include/svtools/inetimg.hxx b/include/svtools/inetimg.hxx
index db061d6a0d4e..2d7f4756f1a0 100644
--- a/include/svtools/inetimg.hxx
+++ b/include/svtools/inetimg.hxx
@@ -56,7 +56,7 @@ public:
     const OUString& GetTargetFrame() const { return aTargetFrame; }
 
     // Im-/Export
-    bool Write( SvStream& rOStm, SotClipboardFormatId nFormat ) const;
+    void Write( SvStream& rOStm, SotClipboardFormatId nFormat ) const;
     bool Read( SvStream& rIStm, SotClipboardFormatId nFormat );
 };
 
diff --git a/include/svtools/ivctrl.hxx b/include/svtools/ivctrl.hxx
index fbd2e599aa3b..98c27ed38107 100644
--- a/include/svtools/ivctrl.hxx
+++ b/include/svtools/ivctrl.hxx
@@ -248,7 +248,7 @@ public:
     void                SetStyle( WinBits nWinStyle );
     WinBits             GetStyle() const;
 
-    bool                SetChoiceWithCursor ( bool bDo = true );
+    void                SetChoiceWithCursor ( bool bDo = true );
 
     void                SetFont( const vcl::Font& rFont );
     void                SetPointFont( const vcl::Font& rFont );
diff --git a/include/svtools/parhtml.hxx b/include/svtools/parhtml.hxx
index 07ee08b5ad52..a32cd0cc399d 100644
--- a/include/svtools/parhtml.hxx
+++ b/include/svtools/parhtml.hxx
@@ -230,7 +230,7 @@ public:
     virtual bool ParseMetaOptions( const css::uno::Reference< css::document::XDocumentProperties>&,
             SvKeyValueIterator* );
 
-    bool ParseScriptOptions( OUString& rLangString, const OUString&, HTMLScriptLanguage& rLang,
+    void ParseScriptOptions( OUString& rLangString, const OUString&, HTMLScriptLanguage& rLang,
                              OUString& rSrc, OUString& rLibrary, OUString& rModule );
 
     // remove a comment around the content of <SCRIPT> or <STYLE>
diff --git a/include/svtools/svlbitm.hxx b/include/svtools/svlbitm.hxx
index 0bbb46da65de..aad018d49dca 100644
--- a/include/svtools/svlbitm.hxx
+++ b/include/svtools/svlbitm.hxx
@@ -268,7 +268,7 @@ public:
     virtual void Clone(SvLBoxItem* pSource) override;
 
 
-    bool SetModeImages(const Image& rBitmap1, const Image& rBitmap2);
+    void SetModeImages(const Image& rBitmap1, const Image& rBitmap2);
 
     inline void SetBitmap1(const Image& rImage);
     inline void SetBitmap2(const Image& rImage);
diff --git a/include/svtools/svparser.hxx b/include/svtools/svparser.hxx
index a91e6a3f0060..3f60a4019613 100644
--- a/include/svtools/svparser.hxx
+++ b/include/svtools/svparser.hxx
@@ -123,10 +123,10 @@ public:
 
     inline sal_uLong    GetLineNr() const       { return nlLineNr; }
     inline sal_uLong    GetLinePos() const      { return nlLinePos; }
-    inline sal_uLong    IncLineNr()             { return ++nlLineNr; }
+    inline void         IncLineNr()             { ++nlLineNr; }
     inline sal_uLong    IncLinePos()            { return ++nlLinePos; }
-    inline sal_uLong    SetLineNr( sal_uLong nlNum );           // inline bottom
-    inline sal_uLong    SetLinePos( sal_uLong nlPos );          // inline bottom
+    inline void         SetLineNr( sal_uLong nlNum );           // inline bottom
+    inline void         SetLinePos( sal_uLong nlPos );          // inline bottom
 
     sal_Unicode GetNextChar();
     void RereadLookahead();
@@ -167,11 +167,11 @@ public:
 };
 
 
-inline sal_uLong SvParser::SetLineNr( sal_uLong nlNum )
-{   sal_uLong nlOld = nlLineNr; nlLineNr = nlNum; return nlOld; }
+inline void SvParser::SetLineNr( sal_uLong nlNum )
+{   nlLineNr = nlNum; }
 
-inline sal_uLong SvParser::SetLinePos( sal_uLong nlPos )
-{   sal_uLong nlOld = nlLinePos; nlLinePos = nlPos; return nlOld; }
+inline void SvParser::SetLinePos( sal_uLong nlPos )
+{   nlLinePos = nlPos; }
 
 inline sal_uInt16 SvParser::GetCharSize() const
 {
diff --git a/include/svtools/tabbar.hxx b/include/svtools/tabbar.hxx
index 29d2e6ba7a91..678981feeb5c 100644
--- a/include/svtools/tabbar.hxx
+++ b/include/svtools/tabbar.hxx
@@ -474,7 +474,7 @@ public:
     bool            StartDrag( const CommandEvent& rCEvt, vcl::Region& rRegion );
     sal_uInt16      ShowDropPos( const Point& rPos );
     void            HideDropPos();
-    bool            SwitchPage( const Point& rPos );
+    void            SwitchPage( const Point& rPos );
     void            EndSwitchPage();
 
     void            SetPageText( sal_uInt16 nPageId, const OUString& rText );
diff --git a/include/svtools/treelistbox.hxx b/include/svtools/treelistbox.hxx
index 9d286ced863c..8bcf3d800395 100644
--- a/include/svtools/treelistbox.hxx
+++ b/include/svtools/treelistbox.hxx
@@ -549,14 +549,14 @@ protected:
     using SvListView::SelectAll;
 
     SVT_DLLPRIVATE short        GetHeightOffset( const Image& rBmp, Size& rLogicSize);
-    SVT_DLLPRIVATE short        GetHeightOffset( const vcl::Font& rFont, Size& rLogicSize);
+    SVT_DLLPRIVATE void         GetHeightOffset( Size& rLogicSize);
 
     SVT_DLLPRIVATE void         SetEntryHeight( SvTreeListEntry* pEntry );
     SVT_DLLPRIVATE void         AdjustEntryHeight( const Image& rBmp );
-    SVT_DLLPRIVATE void         AdjustEntryHeight( const vcl::Font& rFont );
+    SVT_DLLPRIVATE void         AdjustEntryHeight();
 
     SVT_DLLPRIVATE void         ImpEntryInserted( SvTreeListEntry* pEntry );
-    SVT_DLLPRIVATE long         PaintEntry1( SvTreeListEntry&, long nLine, vcl::RenderContext& rRenderContext,
+    SVT_DLLPRIVATE void         PaintEntry1( SvTreeListEntry&, long nLine, vcl::RenderContext& rRenderContext,
                                              SvLBoxTabFlags nTabFlagMask = SvLBoxTabFlags::ALL,
                                              bool bHasClipRegion=false );
 
@@ -581,7 +581,7 @@ protected:
     SvLBoxTab*      GetFirstDynamicTab() const;
     SvLBoxTab*      GetFirstDynamicTab( sal_uInt16& rTabPos ) const;
     SvLBoxTab*      GetFirstTab( SvLBoxTabFlags nFlagMask, sal_uInt16& rTabPos );
-    SvLBoxTab*      GetLastTab( SvLBoxTabFlags nFlagMask, sal_uInt16& rTabPos );
+    void            GetLastTab( SvLBoxTabFlags nFlagMask, sal_uInt16& rTabPos );
     SvLBoxTab*      GetTab( SvTreeListEntry*, SvLBoxItem* ) const;
     void            ClearTabList();
 
@@ -612,7 +612,7 @@ protected:
     bool            AreChildrenTransient() const;
     void            SetChildrenNotTransient();
 
-    void            AdjustEntryHeightAndRecalc( const vcl::Font& rFont );
+    void            AdjustEntryHeightAndRecalc();
 public:
 
     void            SetExtendedWinBits( ExtendedWinBits _nBits );
diff --git a/include/svtools/valueset.hxx b/include/svtools/valueset.hxx
index e4fac32860f0..88da972ee0e8 100644
--- a/include/svtools/valueset.hxx
+++ b/include/svtools/valueset.hxx
@@ -278,7 +278,7 @@ private:
     ValueSet & operator= (const ValueSet &) = delete;
 
 protected:
-    bool StartDrag( const CommandEvent& rCEvt, vcl::Region& rRegion );
+    void StartDrag( const CommandEvent& rCEvt, vcl::Region& rRegion );
 
     virtual css::uno::Reference<css::accessibility::XAccessible> CreateAccessible() override;
 
diff --git a/include/svtools/wizardmachine.hxx b/include/svtools/wizardmachine.hxx
index fab1cfdcbddc..df848febc39c 100644
--- a/include/svtools/wizardmachine.hxx
+++ b/include/svtools/wizardmachine.hxx
@@ -279,13 +279,10 @@ namespace svt
             A very essential precondition for using this method is that your <method>determineNextState</method>
             method is able to determine the next state without actually having the page of the current state.
 
-            @return
-                <TRUE/> if and only if traveling was successful
-
             @see skipUntil
             @see skipBackwardUntil
         */
-        bool                    skip( sal_Int32 _nSteps = 1 );
+        void                    skip( sal_Int32 _nSteps = 1 );
 
         /** skips one or more states, until a given state is reached