loplugin:unusedmethods unused return value in include/svl

Change-Id: I759fe8769daccea07c2157bfb5912df8ba285534

11 files changed, 12 insertions, 45 deletions

diff --git a/include/svl/filerec.hxx b/include/svl/filerec.hxx
index 58271cc50488..b5685f33949f 100644
--- a/include/svl/filerec.hxx
+++ b/include/svl/filerec.hxx
@@ -87,8 +87,6 @@ public:
     inline          SfxMiniRecordWriter( SvStream *pStream, sal_uInt8 nTag );
     inline          ~SfxMiniRecordWriter();
 
-    inline SvStream& operator*() const;
-
     sal_uInt32      Close( bool bSeekToEndOfRec = true );
 
 private:
@@ -257,8 +255,6 @@ public:
     SfxMiniRecordReader( SvStream *pStream, sal_uInt8 nTag );
     inline              ~SfxMiniRecordReader();
 
-    inline SvStream&    operator*() const;
-
     inline void         Skip();
 
 private:
@@ -577,16 +573,6 @@ inline SfxMiniRecordWriter::~SfxMiniRecordWriter()
         Close();
 }
 
-/** Get the record's stream
- * @return The stream containing the record
- * @note The record must not be already closed!
- */
-inline SvStream& SfxMiniRecordWriter::operator*() const
-{
-    DBG_ASSERT( !_bHeaderOk, "getting Stream of closed record" );
-    return *_pStream;
-}
-
 /** The dtor moves the stream automatically to the position directly behind the record */
 inline SfxMiniRecordReader::~SfxMiniRecordReader()
 {
@@ -601,17 +587,6 @@ inline void SfxMiniRecordReader::Skip()
     _bSkipped = true;
 }
 
-/** get the owning stream
- *
- * This method returns the stream in which the record is contained.
- * The current position of the stream must be inside the record.
- */
-inline SvStream& SfxMiniRecordReader::operator*() const
-{
-    DBG_ASSERT( _pStream->Tell() < _nEofRec, "read behind record" );
-    return *_pStream;
-}
-
 /// @see SfxMiniRecordWriter::Close()
 inline sal_uInt32 SfxSingleRecordWriter::Close( bool bSeekToEndOfRec )
 {
diff --git a/include/svl/itemset.hxx b/include/svl/itemset.hxx
index 455be97b809f..16122cec4176 100644
--- a/include/svl/itemset.hxx
+++ b/include/svl/itemset.hxx
@@ -147,9 +147,9 @@ public:
     void                        MergeRange( sal_uInt16 nFrom, sal_uInt16 nTo );
     const SfxItemSet*           GetParent() const { return m_pParent; }
 
-    SvStream &                  Load( SvStream &, bool bDirect = false,
+    void                        Load( SvStream &, bool bDirect = false,
                                       const SfxItemPool *pRefPool = nullptr );
-    SvStream &                  Store( SvStream &, bool bDirect = false ) const;
+    void                        Store( SvStream &, bool bDirect = false ) const;
 
     bool                        operator==(const SfxItemSet &) const;
     sal_Int32                   getHash() const;
diff --git a/include/svl/macitem.hxx b/include/svl/macitem.hxx
index 38babec730ef..a6d78aa638c5 100644
--- a/include/svl/macitem.hxx
+++ b/include/svl/macitem.hxx
@@ -87,7 +87,7 @@ public:
     SvxMacroTableDtor& operator=( const SvxMacroTableDtor &rCpy );
     bool operator==( const SvxMacroTableDtor& rOther ) const;
 
-    SvStream&   Read( SvStream &, sal_uInt16 nVersion = SVX_MACROTBL_AKTVERSION );
+    void        Read( SvStream &, sal_uInt16 nVersion = SVX_MACROTBL_AKTVERSION );
     SvStream&   Write( SvStream & ) const;
 
     static sal_uInt16 GetVersion() { return SVX_MACROTBL_AKTVERSION; }
@@ -108,7 +108,7 @@ public:
     // This stores a copy of the rMacro parameter
     SvxMacro& Insert(sal_uInt16 nEvent, const SvxMacro& rMacro);
     // If the entry exists, remove it from the map and release it's storage
-    bool Erase(sal_uInt16 nEvent);
+    void Erase(sal_uInt16 nEvent);
 };
 
 
diff --git a/include/svl/metitem.hxx b/include/svl/metitem.hxx
index 39ce85159159..7dda419f245e 100644
--- a/include/svl/metitem.hxx
+++ b/include/svl/metitem.hxx
@@ -30,7 +30,7 @@ public:
                              SfxMetricItem( const SfxMetricItem& );
                              virtual ~SfxMetricItem() {}
 
-    virtual bool             ScaleMetrics( long lMult, long lDiv ) override;
+    virtual void             ScaleMetrics( long lMult, long lDiv ) override;
     virtual bool             HasMetrics() const override;
 
 };
diff --git a/include/svl/nranges.hxx b/include/svl/nranges.hxx
index 4cc9fc213f89..def27ace1258 100644
--- a/include/svl/nranges.hxx
+++ b/include/svl/nranges.hxx
@@ -39,8 +39,6 @@ public:
                                 { delete [] _pRanges; }
 
     bool                        operator == ( const SfxUShortRanges & ) const;
-    bool                        operator != ( const SfxUShortRanges & rRanges ) const
-                                { return !( *this == rRanges ); }
 
     SfxUShortRanges&            operator = ( const SfxUShortRanges & );
 
diff --git a/include/svl/ondemand.hxx b/include/svl/ondemand.hxx
index b32dc3608663..cfc729b9b518 100644
--- a/include/svl/ondemand.hxx
+++ b/include/svl/ondemand.hxx
@@ -207,8 +207,6 @@ public:
                                         return pPtr;
                                     }
 
-            CalendarWrapper*    operator->()    { return get(); }
-            CalendarWrapper&    operator*()     { return *get(); }
 };
 
 /** Load a transliteration only if it's needed.
@@ -286,7 +284,6 @@ public:
                                     }
 
     const   ::utl::TransliterationWrapper*  operator->() const  { return get(); }
-    const   ::utl::TransliterationWrapper&  operator*() const   { return *get(); }
 };
 
 /** Load a native number service wrapper only if it's needed.
@@ -340,8 +337,6 @@ public:
                                         return pPtr;
                                     }
 
-            NativeNumberWrapper*    operator->()    { return get(); }
-            NativeNumberWrapper&    operator*()     { return *get(); }
 };
 
 #endif // INCLUDED_SVL_ONDEMAND_HXX
diff --git a/include/svl/ownlist.hxx b/include/svl/ownlist.hxx
index 171d38deab44..460c4a3c0f83 100644
--- a/include/svl/ownlist.hxx
+++ b/include/svl/ownlist.hxx
@@ -62,10 +62,10 @@ private:
     ::std::vector< SvCommand >  aCommandList;
 
 public:
-    SvCommand&      Append( const OUString & rCommand, const OUString & rArg );
-    bool        AppendCommands( const OUString & rCmd, sal_Int32 * pEaten );
+    void           Append( const OUString & rCommand, const OUString & rArg );
+    void           AppendCommands( const OUString & rCmd, sal_Int32 * pEaten );
 
-    bool FillFromSequence( const css::uno::Sequence < css::beans::PropertyValue >& );
+    void FillFromSequence( const css::uno::Sequence < css::beans::PropertyValue >& );
     void FillSequence( css::uno::Sequence < css::beans::PropertyValue >& );
 
     size_t          size() const { return aCommandList.size(); }
diff --git a/include/svl/poolitem.hxx b/include/svl/poolitem.hxx
index 0ae6bea2c512..5d7e4235d6bc 100644
--- a/include/svl/poolitem.hxx
+++ b/include/svl/poolitem.hxx
@@ -177,7 +177,7 @@ public:
                                     const IntlWrapper * pIntlWrapper = nullptr ) const;
 
     virtual sal_uInt16       GetVersion( sal_uInt16 nFileFormatVersion ) const;
-    virtual bool             ScaleMetrics( long lMult, long lDiv );
+    virtual void             ScaleMetrics( long lMult, long lDiv );
     virtual bool             HasMetrics() const;
 
     virtual bool             QueryValue( css::uno::Any& rVal, sal_uInt8 nMemberId = 0 ) const;
diff --git a/include/svl/style.hxx b/include/svl/style.hxx
index 174986e7300c..82173676de2e 100644
--- a/include/svl/style.hxx
+++ b/include/svl/style.hxx
@@ -259,7 +259,7 @@ public:
     /** Add a style sheet.
      * Not an actual public function. Do not call it from non-subclasses.
      */
-    SfxStyleSheetBase&          Add( const SfxStyleSheetBase& );
+    void                        Add( const SfxStyleSheetBase& );
 };
 
 class SVL_DLLPUBLIC SfxStyleSheet: public SfxStyleSheetBase,
diff --git a/include/svl/zforlist.hxx b/include/svl/zforlist.hxx
index 0ca556598823..6178de7eb2d9 100644
--- a/include/svl/zforlist.hxx
+++ b/include/svl/zforlist.hxx
@@ -876,10 +876,9 @@ private:
     // Adjust a sequence of format codes to contain only one (THE) default
     // instead of multiple defaults for short/medium/long types.
     // If there is no medium but a short and a long default the long is taken.
-    // Return the default index in the sequence.
     // Non-PRODUCT version may check locale data for matching defaults in one
     // FormatElement group.
-    SVL_DLLPRIVATE sal_Int32 ImpAdjustFormatCodeDefault( css::i18n::NumberFormatCode * pFormatArr,
+    SVL_DLLPRIVATE void ImpAdjustFormatCodeDefault( css::i18n::NumberFormatCode * pFormatArr,
                                                          sal_Int32 nCount, bool bCheckCorrectness = true );
 
     // Obtain the format entry for a given key index.
diff --git a/include/svl/zformat.hxx b/include/svl/zformat.hxx
index c204702d168c..b66006b1f6fb 100644
--- a/include/svl/zformat.hxx
+++ b/include/svl/zformat.hxx
@@ -208,7 +208,7 @@ public:
     bool GetOutputString( double fNumber, sal_uInt16 nCharCount, OUString& rOutString ) const;
 
     bool GetOutputString( double fNumber, OUString& OutString, Color** ppColor );
-    bool GetOutputString( const OUString& sString, OUString& OutString, Color** ppColor );
+    void GetOutputString( const OUString& sString, OUString& OutString, Color** ppColor );
 
     // True if type text
     bool IsTextFormat() const { return (eType & css::util::NumberFormat::TEXT) != 0; }