loplugin:unusedmethods unused return value in idl

Change-Id: I790fea8b88ca99d17de14f1e91081671422dd8db

7 files changed, 14 insertions, 26 deletions

diff --git a/idl/inc/basobj.hxx b/idl/inc/basobj.hxx
index e3ad1e583585..0725e6d98d01 100644
--- a/idl/inc/basobj.hxx
+++ b/idl/inc/basobj.hxx
@@ -128,7 +128,7 @@ public:
 
     void            Push( SvMetaObject * pObj )
                     { aList.push_back( pObj ); }
-    SvMetaObject *  Pop() { return aList.pop_back(); }
+    void            Pop() { aList.pop_back(); }
     SvMetaObject *  Get( std::function<bool ( const SvMetaObject* )> isSvMetaObject )
                     {
                         for( SvMetaObjectMemberList::reverse_iterator it = aList.rbegin(); it != aList.rend(); ++it )
@@ -200,7 +200,7 @@ public:
 
     SvMetaModule *      GetModule() const;
 
-    const SvGlobalName &GetUUId() const;
+    void                GetUUId() const;
     void                SetModule( SvIdlDataBase & rBase );
     virtual bool        ReadSvIdl( SvIdlDataBase &, SvTokenStream & rInStm ) override;
 
diff --git a/idl/inc/bastype.hxx b/idl/inc/bastype.hxx
index 04335fd0e42c..f31121a78cca 100644
--- a/idl/inc/bastype.hxx
+++ b/idl/inc/bastype.hxx
@@ -79,8 +79,8 @@ public:
     sal_uInt32  GetValue() const { return nValue; }
     void        SetValue( sal_uInt32 nVal ) { nValue = nVal; }
 
-    bool        ReadSvIdl( SvIdlDataBase &, SvTokenStream & rInStm );
-    bool        ReadSvIdl( SvIdlDataBase &, SvStringHashEntry * pName,
+    void        ReadSvIdl( SvIdlDataBase &, SvTokenStream & rInStm );
+    void        ReadSvIdl( SvIdlDataBase &, SvStringHashEntry * pName,
                            SvTokenStream & rInStm );
 };
 
@@ -112,7 +112,7 @@ class SvHelpText : public SvString
 {
 public:
                 SvHelpText() {}
-    bool        ReadSvIdl( SvIdlDataBase &, SvTokenStream & rInStm );
+    void        ReadSvIdl( SvIdlDataBase &, SvTokenStream & rInStm );
 };
 
 
@@ -139,10 +139,6 @@ public:
                     return (r.nMajorVersion == nMajorVersion)
                              && (r.nMinorVersion == nMinorVersion);
                 }
-    bool        operator != ( const SvVersion & r )
-                {
-                    return !(*this == r);
-                }
 
     bool        ReadSvIdl( SvTokenStream & rInStm );
 };
diff --git a/idl/inc/database.hxx b/idl/inc/database.hxx
index 3bf5e0c7a3f1..f7658b2a2ba5 100644
--- a/idl/inc/database.hxx
+++ b/idl/inc/database.hxx
@@ -126,7 +126,7 @@ public:
     SvMetaAttribute *       SearchKnownAttr( const SvIdentifier& );
     SvMetaClass *           ReadKnownClass( SvTokenStream & rInStm );
     void AddDepFile(OUString const& rFileName);
-    bool WriteDepFile(SvFileStream & rStream, OUString const& rTarget);
+    void WriteDepFile(SvFileStream & rStream, OUString const& rTarget);
 };
 
 class SvIdlWorkingBase : public SvIdlDataBase
diff --git a/idl/inc/hash.hxx b/idl/inc/hash.hxx
index 9a3374b20ce6..39fdf767ba0d 100644
--- a/idl/inc/hash.hxx
+++ b/idl/inc/hash.hxx
@@ -79,8 +79,6 @@ public:
 
     bool            operator == ( const SvStringHashEntry & rRef )
                     { return nHashId == rRef.nHashId; }
-    bool            operator != ( const SvStringHashEntry & rRef )
-                    { return ! operator == ( rRef ); }
     SvStringHashEntry & operator = ( const SvStringHashEntry & rRef )
         { SvRefBase::operator=( rRef );
           aName   = rRef.aName;
@@ -110,8 +108,6 @@ public:
     bool    Insert( const OString& rStr, sal_uInt32 * pHash ); // insert string
     bool    Test( const OString& rStr, sal_uInt32 * pHash ) const; // test of insert string
     SvStringHashEntry * Get ( sal_uInt32 nIndex ) const; // return pointer to string
-    SvStringHashEntry & operator []( sal_uInt32 nPos ) const
-            { return pEntries[ nPos ]; }
 };
 
 #endif // INCLUDED_IDL_INC_HASH_HXX
diff --git a/idl/source/objects/basobj.cxx b/idl/source/objects/basobj.cxx
index 4af4327f3e8e..58c08e46ca54 100644
--- a/idl/source/objects/basobj.cxx
+++ b/idl/source/objects/basobj.cxx
@@ -176,11 +176,10 @@ SvMetaModule * SvMetaExtern::GetModule() const
     return pModule;
 }
 
-const SvGlobalName & SvMetaExtern::GetUUId() const
+void SvMetaExtern::GetUUId() const
 {
     if( aUUId == SvGlobalName() )
         GetModule()->FillNextName( &const_cast<SvMetaExtern *>(this)->aUUId );
-    return aUUId;
 }
 
 void SvMetaExtern::SetModule( SvIdlDataBase & rBase )
diff --git a/idl/source/objects/bastype.cxx b/idl/source/objects/bastype.cxx
index 2f7adacef928..40b317e58443 100644
--- a/idl/source/objects/bastype.cxx
+++ b/idl/source/objects/bastype.cxx
@@ -115,7 +115,7 @@ bool SvIdentifier::ReadSvIdl( SvStringHashEntry * pName, SvTokenStream & rInStm
     return false;
 }
 
-bool SvIdentifier::ReadSvIdl( SvIdlDataBase & rBase,
+void SvIdentifier::ReadSvIdl( SvIdlDataBase & rBase,
                                     SvStringHashEntry * pName,
                                     SvTokenStream & rInStm )
 {
@@ -125,7 +125,7 @@ bool SvIdentifier::ReadSvIdl( SvIdlDataBase & rBase,
         if( rBase.FindId( getString(), &n ) )
         {
             nValue = n;
-            return true;
+            return;
         }
         else
         {
@@ -135,10 +135,9 @@ bool SvIdentifier::ReadSvIdl( SvIdlDataBase & rBase,
             rBase.WriteError( rInStm );
         }
     }
-    return false;
 }
 
-bool SvIdentifier::ReadSvIdl( SvIdlDataBase & rBase,
+void SvIdentifier::ReadSvIdl( SvIdlDataBase & rBase,
                                     SvTokenStream & rInStm )
 {
     sal_uInt32 nTokPos = rInStm.Tell();
@@ -151,7 +150,7 @@ bool SvIdentifier::ReadSvIdl( SvIdlDataBase & rBase,
         {
             setString(pTok->GetString());
             nValue = n;
-            return true;
+            return;
         }
         else
         {
@@ -162,7 +161,6 @@ bool SvIdentifier::ReadSvIdl( SvIdlDataBase & rBase,
         }
     }
     rInStm.Seek( nTokPos );
-    return false;
 }
 
 bool SvString::ReadSvIdl( SvStringHashEntry * pName, SvTokenStream & rInStm )
@@ -192,9 +190,9 @@ bool SvString::ReadSvIdl( SvStringHashEntry * pName, SvTokenStream & rInStm )
     return false;
 }
 
-bool SvHelpText::ReadSvIdl( SvIdlDataBase &, SvTokenStream & rInStm )
+void SvHelpText::ReadSvIdl( SvIdlDataBase &, SvTokenStream & rInStm )
 {
-    return SvString::ReadSvIdl( SvHash_HelpText(), rInStm );
+    SvString::ReadSvIdl( SvHash_HelpText(), rInStm );
 }
 
 bool SvUUId::ReadSvIdl( SvIdlDataBase &, SvTokenStream & rInStm )
diff --git a/idl/source/prj/database.cxx b/idl/source/prj/database.cxx
index 48a3f5ea53f9..7703530f3097 100644
--- a/idl/source/prj/database.cxx
+++ b/idl/source/prj/database.cxx
@@ -674,7 +674,7 @@ struct WriteDummy
     }
 };
 
-bool SvIdlDataBase::WriteDepFile(
+void SvIdlDataBase::WriteDepFile(
         SvFileStream & rStream, OUString const& rTarget)
 {
     rStream.WriteCharPtr( OUStringToOString(rTarget, RTL_TEXTENCODING_UTF8).getStr() );
@@ -682,7 +682,6 @@ bool SvIdlDataBase::WriteDepFile(
     ::std::for_each(m_DepFiles.begin(), m_DepFiles.end(), WriteDep(rStream));
     rStream.WriteCharPtr( "\n\n" );
     ::std::for_each(m_DepFiles.begin(), m_DepFiles.end(), WriteDummy(rStream));
-    return rStream.GetError() == SVSTREAM_OK;
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */