loplugin:casttovoid: basic

Change-Id: I4e70accf67d4d812b8998b0baa07cd04c27216ad

18 files changed, 104 insertions, 211 deletions

diff --git a/basic/inc/sbstdobj.hxx b/basic/inc/sbstdobj.hxx
index d306c3d81e97..210d3f7a26c4 100644
--- a/basic/inc/sbstdobj.hxx
+++ b/basic/inc/sbstdobj.hxx
@@ -102,11 +102,11 @@ protected:
     virtual void   Notify( SfxBroadcaster& rBC, const SfxHint& rHint ) override;
 
     static void    MethClear( SbxVariable* pVar, SbxArray* pPar_, bool bWrite );
-    static void    MethGetData( SbxVariable* pVar, SbxArray* pPar_, bool bWrite );
+    static void    MethGetData( SbxArray* pPar_, bool bWrite );
     static void    MethGetFormat( SbxVariable* pVar, SbxArray* pPar_, bool bWrite );
     static void    MethGetText( SbxVariable* pVar, SbxArray* pPar_, bool bWrite );
-    static void    MethSetData( SbxVariable* pVar, SbxArray* pPar_, bool bWrite );
-    static void    MethSetText( SbxVariable* pVar, SbxArray* pPar_, bool bWrite );
+    static void    MethSetData( SbxArray* pPar_, bool bWrite );
+    static void    MethSetText( SbxArray* pPar_, bool bWrite );
 
 public:
 
diff --git a/basic/source/basmgr/basmgr.cxx b/basic/source/basmgr/basmgr.cxx
index 2b71f0439003..762ac7a70ef3 100644
--- a/basic/source/basmgr/basmgr.cxx
+++ b/basic/source/basmgr/basmgr.cxx
@@ -212,10 +212,7 @@ void BasMgrContainerListenerImpl::addLibraryModulesImpl( BasicManager* pMgr,
 // XEventListener
 
 
-void SAL_CALL BasMgrContainerListenerImpl::disposing( const lang::EventObject& Source )
-{
-    (void)Source;
-}
+void SAL_CALL BasMgrContainerListenerImpl::disposing( const lang::EventObject& ) {}
 
 // XContainerListener
 
@@ -1926,9 +1923,8 @@ void DialogContainer_Impl::replaceByName( const OUString& aName, const uno::Any&
 
 
 // Methods XNameContainer
-void DialogContainer_Impl::insertByName( const OUString& aName, const uno::Any& aElement )
+void DialogContainer_Impl::insertByName( const OUString&, const uno::Any& aElement )
 {
-    (void)aName;
     uno::Type aModuleType = cppu::UnoType<script::XStarBasicDialogInfo>::get();
     const uno::Type& aAnyType = aElement.getValueType();
     if( aModuleType != aAnyType )
@@ -1943,7 +1939,6 @@ void DialogContainer_Impl::insertByName( const OUString& aName, const uno::Any&
 
 void DialogContainer_Impl::removeByName( const OUString& Name )
 {
-    (void)Name;
     SbxVariable* pVar = mpLib->GetObjects()->Find( Name, SbxClassType::DontCare );
     SbxObject* pObj = dynamic_cast<SbxObject*>(pVar);
     if( !( pObj && ( pObj->GetSbxId() == SBXID_DIALOG ) ) )
@@ -2063,10 +2058,8 @@ void LibraryContainer_Impl::replaceByName( const OUString& aName, const uno::Any
 }
 
 // Methods XNameContainer
-void LibraryContainer_Impl::insertByName( const OUString& aName, const uno::Any& aElement )
+void LibraryContainer_Impl::insertByName( const OUString&, const uno::Any& )
 {
-    (void)aName;
-    (void)aElement;
     // TODO: Insert a complete Library?!
 }
 
@@ -2116,11 +2109,10 @@ void SAL_CALL StarBasicAccess_Impl::createLibrary
 (
     const OUString& LibName,
     const OUString& Password,
-    const OUString& ExternalSourceURL,
+    const OUString&,
     const OUString& LinkTargetURL
 )
 {
-    (void)ExternalSourceURL;
     StarBASIC* pLib = mpMgr->CreateLib( LibName, Password, LinkTargetURL );
     DBG_ASSERT( pLib, "XML Import: Basic library could not be created");
 }
@@ -2129,11 +2121,10 @@ void SAL_CALL StarBasicAccess_Impl::addModule
 (
     const OUString& LibraryName,
     const OUString& ModuleName,
-    const OUString& Language,
+    const OUString&,
     const OUString& Source
 )
 {
-    (void)Language;
     StarBASIC* pLib = mpMgr->GetLib( LibraryName );
     DBG_ASSERT( pLib, "XML Import: Lib for module unknown");
     if( pLib )
@@ -2144,15 +2135,11 @@ void SAL_CALL StarBasicAccess_Impl::addModule
 
 void SAL_CALL StarBasicAccess_Impl::addDialog
 (
-    const OUString& LibraryName,
-    const OUString& DialogName,
-    const uno::Sequence< sal_Int8 >& Data
+    const OUString&,
+    const OUString&,
+    const uno::Sequence< sal_Int8 >&
 )
-{
-    (void)LibraryName;
-    (void)DialogName;
-    (void)Data;
-}
+{}
 
 // Basic XML Import/Export
 uno::Reference< script::XStarBasicAccess > getStarBasicAccess( BasicManager* pMgr )
diff --git a/basic/source/classes/eventatt.cxx b/basic/source/classes/eventatt.cxx
index 1eb725e0c176..bd81d108cfb6 100644
--- a/basic/source/classes/eventatt.cxx
+++ b/basic/source/classes/eventatt.cxx
@@ -401,11 +401,8 @@ css::uno::Reference< css::container::XNameContainer > implFindDialogLibForDialog
     return aDlgLib;
 }
 
-void RTL_Impl_CreateUnoDialog( StarBASIC* pBasic, SbxArray& rPar, bool bWrite )
+void RTL_Impl_CreateUnoDialog( SbxArray& rPar )
 {
-    (void)pBasic;
-    (void)bWrite;
-
     Reference< XComponentContext > xContext( comphelper::getProcessComponentContext() );
 
     // We need at least 1 parameter
diff --git a/basic/source/classes/propacc.cxx b/basic/source/classes/propacc.cxx
index 1dc958879ddf..0e9cd5e5298c 100644
--- a/basic/source/classes/propacc.cxx
+++ b/basic/source/classes/propacc.cxx
@@ -112,35 +112,27 @@ Any SbPropertyValues::getPropertyValue(
 
 
 void SbPropertyValues::addPropertyChangeListener(
-                    const OUString& aPropertyName,
+                    const OUString&,
                     const Reference< XPropertyChangeListener >& )
-{
-    (void)aPropertyName;
-}
+{}
 
 
 void SbPropertyValues::removePropertyChangeListener(
-                    const OUString& aPropertyName,
+                    const OUString&,
                     const Reference< XPropertyChangeListener >& )
-{
-    (void)aPropertyName;
-}
+{}
 
 
 void SbPropertyValues::addVetoableChangeListener(
-                    const OUString& aPropertyName,
+                    const OUString&,
                     const Reference< XVetoableChangeListener >& )
-{
-    (void)aPropertyName;
-}
+{}
 
 
 void SbPropertyValues::removeVetoableChangeListener(
-                    const OUString& aPropertyName,
+                    const OUString&,
                     const Reference< XVetoableChangeListener >& )
-{
-    (void)aPropertyName;
-}
+{}
 
 
 Sequence< PropertyValue > SbPropertyValues::getPropertyValues()
@@ -162,11 +154,8 @@ void SbPropertyValues::setPropertyValues(const Sequence< PropertyValue >& rPrope
 }
 
 
-void RTL_Impl_CreatePropertySet( StarBASIC* pBasic, SbxArray& rPar, bool bWrite )
+void RTL_Impl_CreatePropertySet( SbxArray& rPar )
 {
-    (void)pBasic;
-    (void)bWrite;
-
     // We need at least one parameter
     // TODO: In this case < 2 is not correct ;-)
     if ( rPar.Count() < 2 )
diff --git a/basic/source/classes/sbunoobj.cxx b/basic/source/classes/sbunoobj.cxx
index bc70879aecea..0a74cf022eea 100644
--- a/basic/source/classes/sbunoobj.cxx
+++ b/basic/source/classes/sbunoobj.cxx
@@ -2902,11 +2902,8 @@ void createAllObjectProperties( SbxObject* pObj )
 }
 
 
-void RTL_Impl_CreateUnoStruct( StarBASIC* pBasic, SbxArray& rPar, bool bWrite )
+void RTL_Impl_CreateUnoStruct( SbxArray& rPar )
 {
-    (void)pBasic;
-    (void)bWrite;
-
     // We need 1 parameter minimum
     if ( rPar.Count() < 2 )
     {
@@ -2928,11 +2925,8 @@ void RTL_Impl_CreateUnoStruct( StarBASIC* pBasic, SbxArray& rPar, bool bWrite )
     refVar->PutObject( xUnoObj.get() );
 }
 
-void RTL_Impl_CreateUnoService( StarBASIC* pBasic, SbxArray& rPar, bool bWrite )
+void RTL_Impl_CreateUnoService( SbxArray& rPar )
 {
-    (void)pBasic;
-    (void)bWrite;
-
     // We need 1 Parameter minimum
     if ( rPar.Count() < 2 )
     {
@@ -2976,11 +2970,8 @@ void RTL_Impl_CreateUnoService( StarBASIC* pBasic, SbxArray& rPar, bool bWrite )
     }
 }
 
-void RTL_Impl_CreateUnoServiceWithArguments( StarBASIC* pBasic, SbxArray& rPar, bool bWrite )
+void RTL_Impl_CreateUnoServiceWithArguments( SbxArray& rPar )
 {
-    (void)pBasic;
-    (void)bWrite;
-
     // We need 2 parameter minimum
     if ( rPar.Count() < 3 )
     {
@@ -3028,11 +3019,8 @@ void RTL_Impl_CreateUnoServiceWithArguments( StarBASIC* pBasic, SbxArray& rPar,
     }
 }
 
-void RTL_Impl_GetProcessServiceManager( StarBASIC* pBasic, SbxArray& rPar, bool bWrite )
+void RTL_Impl_GetProcessServiceManager( SbxArray& rPar )
 {
-    (void)pBasic;
-    (void)bWrite;
-
     SbxVariableRef refVar = rPar.Get(0);
 
     // get the global service manager
@@ -3043,11 +3031,8 @@ void RTL_Impl_GetProcessServiceManager( StarBASIC* pBasic, SbxArray& rPar, bool
     refVar->PutObject( xUnoObj.get() );
 }
 
-void RTL_Impl_HasInterfaces( StarBASIC* pBasic, SbxArray& rPar, bool bWrite )
+void RTL_Impl_HasInterfaces( SbxArray& rPar )
 {
-    (void)pBasic;
-    (void)bWrite;
-
     // We need 2 parameter minimum
     sal_uInt16 nParCount = rPar.Count();
     if( nParCount < 3 )
@@ -3104,11 +3089,8 @@ void RTL_Impl_HasInterfaces( StarBASIC* pBasic, SbxArray& rPar, bool bWrite )
     refVar->PutBool( true );
 }
 
-void RTL_Impl_IsUnoStruct( StarBASIC* pBasic, SbxArray& rPar, bool bWrite )
+void RTL_Impl_IsUnoStruct( SbxArray& rPar )
 {
-    (void)pBasic;
-    (void)bWrite;
-
     // We need 1 parameter minimum
     if ( rPar.Count() < 2 )
     {
@@ -3141,11 +3123,8 @@ void RTL_Impl_IsUnoStruct( StarBASIC* pBasic, SbxArray& rPar, bool bWrite )
 }
 
 
-void RTL_Impl_EqualUnoObjects( StarBASIC* pBasic, SbxArray& rPar, bool bWrite )
+void RTL_Impl_EqualUnoObjects( SbxArray& rPar )
 {
-    (void)pBasic;
-    (void)bWrite;
-
     if ( rPar.Count() < 3 )
     {
         StarBASIC::Error( ERRCODE_BASIC_BAD_ARGUMENT );
@@ -3963,11 +3942,8 @@ Reference< XIntrospectionAccess > SAL_CALL InvocationToAllListenerMapper::getInt
 
 
 Any SAL_CALL InvocationToAllListenerMapper::invoke(const OUString& FunctionName, const Sequence< Any >& Params,
-    Sequence< sal_Int16 >& OutParamIndex, Sequence< Any >& OutParam)
+    Sequence< sal_Int16 >&, Sequence< Any >&)
 {
-    (void)OutParamIndex;
-    (void)OutParam     ;
-
     Any aRet;
 
     // Check if to firing or approveFiring has to be called
@@ -4014,17 +3990,12 @@ Any SAL_CALL InvocationToAllListenerMapper::invoke(const OUString& FunctionName,
 }
 
 
-void SAL_CALL InvocationToAllListenerMapper::setValue(const OUString& PropertyName, const Any& Value)
-{
-    (void)PropertyName;
-    (void)Value;
-}
+void SAL_CALL InvocationToAllListenerMapper::setValue(const OUString&, const Any&)
+{}
 
 
-Any SAL_CALL InvocationToAllListenerMapper::getValue(const OUString& PropertyName)
+Any SAL_CALL InvocationToAllListenerMapper::getValue(const OUString&)
 {
-    (void)PropertyName;
-
     return Any();
 }
 
@@ -4107,11 +4078,8 @@ RTLFUNC(CreateUnoListener)
 
 // Represents the DefaultContext property of the ProcessServiceManager
 // in the Basic runtime system.
-void RTL_Impl_GetDefaultContext( StarBASIC* pBasic, SbxArray& rPar, bool bWrite )
+void RTL_Impl_GetDefaultContext( SbxArray& rPar )
 {
-    (void)pBasic;
-    (void)bWrite;
-
     SbxVariableRef refVar = rPar.Get(0);
 
     Any aContextAny( comphelper::getProcessComponentContext() );
@@ -4123,11 +4091,8 @@ void RTL_Impl_GetDefaultContext( StarBASIC* pBasic, SbxArray& rPar, bool bWrite
 
 // Creates a Basic wrapper object for a strongly typed Uno value
 // 1. parameter: Uno type as full qualified type name, e.g. "byte[]"
-void RTL_Impl_CreateUnoValue( StarBASIC* pBasic, SbxArray& rPar, bool bWrite )
+void RTL_Impl_CreateUnoValue( SbxArray& rPar )
 {
-    (void)pBasic;
-    (void)bWrite;
-
     // 2 parameters needed
     if ( rPar.Count() != 3 )
     {
diff --git a/basic/source/classes/sbxmod.cxx b/basic/source/classes/sbxmod.cxx
index d56894fffe3d..61e1cfd7da6e 100644
--- a/basic/source/classes/sbxmod.cxx
+++ b/basic/source/classes/sbxmod.cxx
@@ -1344,8 +1344,6 @@ void SbModule::implClearIfVarDependsOnDeletedBasic( SbxVariable* pVar, StarBASIC
 
 void SbModule::ClearVarsDependingOnDeletedBasic( StarBASIC* pDeletedBasic )
 {
-    (void)pDeletedBasic;
-
     for( sal_uInt16 i = 0 ; i < pProps->Count() ; i++ )
     {
         SbProperty* p = dynamic_cast<SbProperty*>( pProps->Get( i )  );
@@ -1915,10 +1913,8 @@ SbJScriptModule::SbJScriptModule()
 {
 }
 
-bool SbJScriptModule::LoadData( SvStream& rStrm, sal_uInt16 nVer )
+bool SbJScriptModule::LoadData( SvStream& rStrm, sal_uInt16 )
 {
-    (void)nVer;
-
     Clear();
     if( !SbxObject::LoadData( rStrm, 1 ) )
         return false;
diff --git a/basic/source/comp/dim.cxx b/basic/source/comp/dim.cxx
index a5d5380a53ed..73862b3bf26a 100644
--- a/basic/source/comp/dim.cxx
+++ b/basic/source/comp/dim.cxx
@@ -271,7 +271,7 @@ void SbiParser::DefVar( SbiOpcode eOp, bool bStatic )
         else if( eCurTok == TYPE )
         {
             Next();
-            DefType( bPrivate );
+            DefType(); // TODO: Use bPrivate in DefType()
             return;
         }
     }
@@ -562,14 +562,11 @@ void SbiParser::Erase()
 
 void SbiParser::Type()
 {
-    DefType( false );
+    DefType();
 }
 
-void SbiParser::DefType( bool bPrivate )
+void SbiParser::DefType()
 {
-    // TODO: Use bPrivate
-    (void)bPrivate;
-
     // Read the new Token lesen. It had to be a symbol
     if (!TestSymbol())
         return;
diff --git a/basic/source/inc/eventatt.hxx b/basic/source/inc/eventatt.hxx
index 881290814787..fbe0104db00f 100644
--- a/basic/source/inc/eventatt.hxx
+++ b/basic/source/inc/eventatt.hxx
@@ -23,12 +23,11 @@
 #include <sal/config.h>
 
 class SbxArray;
-class StarBASIC;
 
 // Instantiate "com.sun.star.awt.UnoControlDialog" on basis
 // of a DialogLibrary entry: Convert from XML-ByteSequence
 // and attach events.
-void RTL_Impl_CreateUnoDialog( StarBASIC* pBasic, SbxArray& rPar, bool bWrite );
+void RTL_Impl_CreateUnoDialog( SbxArray& rPar );
 
 #endif
 
diff --git a/basic/source/inc/parser.hxx b/basic/source/inc/parser.hxx
index a2138283bb22..4ff5baf83518 100644
--- a/basic/source/inc/parser.hxx
+++ b/basic/source/inc/parser.hxx
@@ -54,7 +54,7 @@ class SbiParser : public SbiTokenizer
     void CloseBlock();
     bool Channel( bool bAlways=false );     // parse channel number
     void StmntBlock( SbiToken );
-    void DefType( bool bPrivate );  // Parse type declaration
+    void DefType();  // Parse type declaration
     void DefEnum( bool bPrivate );  // Parse enum declaration
     void DefDeclare( bool bPrivate );
     void EnableCompatibility();
diff --git a/basic/source/inc/propacc.hxx b/basic/source/inc/propacc.hxx
index e1fd4802ce12..0e174b2ef713 100644
--- a/basic/source/inc/propacc.hxx
+++ b/basic/source/inc/propacc.hxx
@@ -71,10 +71,9 @@ public:
     virtual void SAL_CALL setPropertyValues(const css::uno::Sequence< css::beans::PropertyValue >& PropertyValues_) override;
 };
 
-class StarBASIC;
 class SbxArray;
 
-void RTL_Impl_CreatePropertySet( StarBASIC* pBasic, SbxArray& rPar, bool bWrite );
+void RTL_Impl_CreatePropertySet( SbxArray& rPar );
 
 #endif
 
diff --git a/basic/source/inc/sbunoobj.hxx b/basic/source/inc/sbunoobj.hxx
index a707d72a1d1e..bbb8f08f42cd 100644
--- a/basic/source/inc/sbunoobj.hxx
+++ b/basic/source/inc/sbunoobj.hxx
@@ -327,15 +327,15 @@ public:
 class StarBASIC;
 
 // Impl-methods for RTL
-void RTL_Impl_CreateUnoStruct( StarBASIC* pBasic, SbxArray& rPar, bool bWrite );
-void RTL_Impl_CreateUnoService( StarBASIC* pBasic, SbxArray& rPar, bool bWrite );
-void RTL_Impl_CreateUnoServiceWithArguments( StarBASIC* pBasic, SbxArray& rPar, bool bWrite );
-void RTL_Impl_CreateUnoValue( StarBASIC* pBasic, SbxArray& rPar, bool bWrite );
-void RTL_Impl_GetProcessServiceManager( StarBASIC* pBasic, SbxArray& rPar, bool bWrite );
-void RTL_Impl_HasInterfaces( StarBASIC* pBasic, SbxArray& rPar, bool bWrite );
-void RTL_Impl_IsUnoStruct( StarBASIC* pBasic, SbxArray& rPar, bool bWrite );
-void RTL_Impl_EqualUnoObjects( StarBASIC* pBasic, SbxArray& rPar, bool bWrite );
-void RTL_Impl_GetDefaultContext( StarBASIC* pBasic, SbxArray& rPar, bool bWrite );
+void RTL_Impl_CreateUnoStruct( SbxArray& rPar );
+void RTL_Impl_CreateUnoService( SbxArray& rPar );
+void RTL_Impl_CreateUnoServiceWithArguments( SbxArray& rPar );
+void RTL_Impl_CreateUnoValue( SbxArray& rPar );
+void RTL_Impl_GetProcessServiceManager( SbxArray& rPar );
+void RTL_Impl_HasInterfaces( SbxArray& rPar );
+void RTL_Impl_IsUnoStruct( SbxArray& rPar );
+void RTL_Impl_EqualUnoObjects( SbxArray& rPar );
+void RTL_Impl_GetDefaultContext( SbxArray& rPar );
 
 void disposeComVariablesForBasic( StarBASIC* pBasic );
 void clearNativeObjectWrapperVector();
diff --git a/basic/source/runtime/iosys.cxx b/basic/source/runtime/iosys.cxx
index dce5777e14af..11c2806a4632 100644
--- a/basic/source/runtime/iosys.cxx
+++ b/basic/source/runtime/iosys.cxx
@@ -438,10 +438,8 @@ void UCBStream::FlushData()
     }
 }
 
-void    UCBStream::SetSize( sal_uInt64 nSize )
+void    UCBStream::SetSize( sal_uInt64 )
 {
-    (void)nSize;
-
     SAL_WARN("basic", "UCBStream::SetSize not allowed to call from basic" );
     SetError( ERRCODE_IO_GENERAL );
 }
diff --git a/basic/source/runtime/methods.cxx b/basic/source/runtime/methods.cxx
index 1ca9282f5468..aca8c7be6ae7 100644
--- a/basic/source/runtime/methods.cxx
+++ b/basic/source/runtime/methods.cxx
@@ -608,9 +608,6 @@ RTLFUNC(Kill)
 
 RTLFUNC(MkDir)
 {
-    (void)pBasic;
-    (void)bWrite;
-
     rPar.Get(0)->PutEmpty();
     if (rPar.Count() == 2)
     {
@@ -1182,7 +1179,6 @@ RTLFUNC(LTrim)
 RTLFUNC(Mid)
 {
     (void)pBasic;
-    (void)bWrite;
 
     int nArgCount = rPar.Count()-1;
     if ( nArgCount < 2 )
@@ -1450,7 +1446,6 @@ RTLFUNC(Right)
 
 RTLFUNC(RTL)
 {
-    (void)pBasic;
     (void)bWrite;
 
     rPar.Get( 0 )->PutObject( pBasic->getRTL().get() );
@@ -2521,7 +2516,6 @@ RTLFUNC(Timer)
 RTLFUNC(Date)
 {
     (void)pBasic;
-    (void)bWrite;
 
     if ( !bWrite )
     {
@@ -4735,7 +4729,6 @@ RTLFUNC(Reset)
 
 RTLFUNC(DumpAllObjects)
 {
-    (void)pBasic;
     (void)bWrite;
 
     sal_uInt16 nArgCount = rPar.Count();
diff --git a/basic/source/runtime/methods1.cxx b/basic/source/runtime/methods1.cxx
index d6badbf225c9..4344dfaacf17 100644
--- a/basic/source/runtime/methods1.cxx
+++ b/basic/source/runtime/methods1.cxx
@@ -670,7 +670,6 @@ RTLFUNC(DoEvents)
 {
     (void)pBasic;
     (void)bWrite;
-    (void)rPar;
 // don't understand what upstream are up to
 // we already process application events etc. in between
 // basic runtime pcode ( on a timed basis )
@@ -1084,11 +1083,8 @@ static bool lcl_WriteSbxVariable( const SbxVariable& rVar, SvStream* pStrm,
 }
 
 static bool lcl_ReadSbxVariable( SbxVariable& rVar, SvStream* pStrm,
-                                     bool bBinary, short nBlockLen, bool bIsArray )
+                                     bool bBinary, short nBlockLen )
 {
-    (void)bBinary;
-    (void)bIsArray;
-
     double aDouble;
 
     sal_uInt64 const nFPos = pStrm->Tell();
@@ -1215,7 +1211,7 @@ static bool lcl_WriteReadSbxArray( SbxDimArray& rArr, SvStream* pStrm,
             if( bWrite )
                 bRet = lcl_WriteSbxVariable(*pVar, pStrm, bBinary, 0, true );
             else
-                bRet = lcl_ReadSbxVariable(*pVar, pStrm, bBinary, 0, true );
+                bRet = lcl_ReadSbxVariable(*pVar, pStrm, bBinary, 0 );
             if( !bRet )
                 return false;
         }
@@ -1292,7 +1288,7 @@ void PutGet( SbxArray& rPar, bool bPut )
         if( bPut )
             bRet = lcl_WriteSbxVariable(*pVar, pStrm, !bRandom, nBlockLen, false);
         else
-            bRet = lcl_ReadSbxVariable(*pVar, pStrm, !bRandom, nBlockLen, false);
+            bRet = lcl_ReadSbxVariable(*pVar, pStrm, !bRandom, nBlockLen);
     }
     if( !bRet || pStrm->GetErrorCode() )
         StarBASIC::Error( ERRCODE_BASIC_IO_ERROR );
@@ -1532,7 +1528,7 @@ RTLFUNC(CreateUnoStruct)
     (void)pBasic;
     (void)bWrite;
 
-    RTL_Impl_CreateUnoStruct( pBasic, rPar, bWrite );
+    RTL_Impl_CreateUnoStruct( rPar );
 }
 
 
@@ -1542,7 +1538,7 @@ RTLFUNC(CreateUnoService)
     (void)pBasic;
     (void)bWrite;
 
-    RTL_Impl_CreateUnoService( pBasic, rPar, bWrite );
+    RTL_Impl_CreateUnoService( rPar );
 }
 
 RTLFUNC(CreateUnoServiceWithArguments)
@@ -1550,7 +1546,7 @@ RTLFUNC(CreateUnoServiceWithArguments)
     (void)pBasic;
     (void)bWrite;
 
-    RTL_Impl_CreateUnoServiceWithArguments( pBasic, rPar, bWrite );
+    RTL_Impl_CreateUnoServiceWithArguments( rPar );
 }
 
 
@@ -1559,7 +1555,7 @@ RTLFUNC(CreateUnoValue)
     (void)pBasic;
     (void)bWrite;
 
-    RTL_Impl_CreateUnoValue( pBasic, rPar, bWrite );
+    RTL_Impl_CreateUnoValue( rPar );
 }
 
 
@@ -1569,7 +1565,7 @@ RTLFUNC(GetProcessServiceManager)
     (void)pBasic;
     (void)bWrite;
 
-    RTL_Impl_GetProcessServiceManager( pBasic, rPar, bWrite );
+    RTL_Impl_GetProcessServiceManager( rPar );
 }
 
 
@@ -1579,7 +1575,7 @@ RTLFUNC(CreatePropertySet)
     (void)pBasic;
     (void)bWrite;
 
-    RTL_Impl_CreatePropertySet( pBasic, rPar, bWrite );
+    RTL_Impl_CreatePropertySet( rPar );
 }
 
 
@@ -1589,7 +1585,7 @@ RTLFUNC(HasUnoInterfaces)
     (void)pBasic;
     (void)bWrite;
 
-    RTL_Impl_HasInterfaces( pBasic, rPar, bWrite );
+    RTL_Impl_HasInterfaces( rPar );
 }
 
 
@@ -1598,7 +1594,7 @@ RTLFUNC(IsUnoStruct)
     (void)pBasic;
     (void)bWrite;
 
-    RTL_Impl_IsUnoStruct( pBasic, rPar, bWrite );
+    RTL_Impl_IsUnoStruct( rPar );
 }
 
 
@@ -1607,7 +1603,7 @@ RTLFUNC(EqualUnoObjects)
     (void)pBasic;
     (void)bWrite;
 
-    RTL_Impl_EqualUnoObjects( pBasic, rPar, bWrite );
+    RTL_Impl_EqualUnoObjects( rPar );
 }
 
 RTLFUNC(CreateUnoDialog)
@@ -1615,13 +1611,12 @@ RTLFUNC(CreateUnoDialog)
     (void)pBasic;
     (void)bWrite;
 
-    RTL_Impl_CreateUnoDialog( pBasic, rPar, bWrite );
+    RTL_Impl_CreateUnoDialog( rPar );
 }
 
 // Return the application standard lib as root scope
 RTLFUNC(GlobalScope)
 {
-    (void)pBasic;
     (void)bWrite;
 
     SbxObject* p = pBasic;
@@ -1689,7 +1684,7 @@ RTLFUNC(GetDefaultContext)
     (void)pBasic;
     (void)bWrite;
 
-    RTL_Impl_GetDefaultContext( pBasic, rPar, bWrite );
+    RTL_Impl_GetDefaultContext( rPar );
 }
 
 RTLFUNC(Join)
diff --git a/basic/source/runtime/props.cxx b/basic/source/runtime/props.cxx
index ce994947389e..7c77a34ceb0d 100644
--- a/basic/source/runtime/props.cxx
+++ b/basic/source/runtime/props.cxx
@@ -40,7 +40,6 @@ RTLFUNC(Erl)
 RTLFUNC(Err)
 {
     (void)pBasic;
-    (void)bWrite;
 
     if( SbiRuntime::isVBAEnabled() )
     {
diff --git a/basic/source/runtime/stdobj1.cxx b/basic/source/runtime/stdobj1.cxx
index 861617b7df77..d0dedad8961f 100644
--- a/basic/source/runtime/stdobj1.cxx
+++ b/basic/source/runtime/stdobj1.cxx
@@ -287,10 +287,8 @@ void SbStdClipboard::MethClear( SbxVariable*, SbxArray* pPar_, bool )
 
 }
 
-void SbStdClipboard::MethGetData( SbxVariable* pVar, SbxArray* pPar_, bool )
+void SbStdClipboard::MethGetData( SbxArray* pPar_, bool )
 {
-    (void)pVar;
-
     if( !pPar_ || (pPar_->Count() != 2) )
     {
         StarBASIC::Error( ERRCODE_BASIC_BAD_NUMBER_OF_ARGS );
@@ -335,10 +333,8 @@ void SbStdClipboard::MethGetText( SbxVariable* pVar, SbxArray* pPar_, bool )
     pVar->PutString( OUString() );
 }
 
-void SbStdClipboard::MethSetData( SbxVariable* pVar, SbxArray* pPar_, bool )
+void SbStdClipboard::MethSetData( SbxArray* pPar_, bool )
 {
-    (void)pVar;
-
     if( !pPar_ || (pPar_->Count() != 3) )
     {
         StarBASIC::Error( ERRCODE_BASIC_BAD_NUMBER_OF_ARGS );
@@ -354,10 +350,8 @@ void SbStdClipboard::MethSetData( SbxVariable* pVar, SbxArray* pPar_, bool )
 
 }
 
-void SbStdClipboard::MethSetText( SbxVariable* pVar, SbxArray* pPar_, bool )
+void SbStdClipboard::MethSetText( SbxArray* pPar_, bool )
 {
-    (void)pVar;
-
     if( !pPar_ || (pPar_->Count() != 2) )
     {
         StarBASIC::Error( ERRCODE_BASIC_BAD_NUMBER_OF_ARGS );
@@ -420,11 +414,11 @@ void SbStdClipboard::Notify( SfxBroadcaster& rBC, const SfxHint& rHint )
         switch( nWhich )
         {
             case METH_CLEAR:            MethClear( pVar, pPar_, bWrite ); return;
-            case METH_GETDATA:          MethGetData( pVar, pPar_, bWrite ); return;
+            case METH_GETDATA:          MethGetData( pPar_, bWrite ); return;
             case METH_GETFORMAT:        MethGetFormat( pVar, pPar_, bWrite ); return;
             case METH_GETTEXT:          MethGetText( pVar, pPar_, bWrite ); return;
-            case METH_SETDATA:          MethSetData( pVar, pPar_, bWrite ); return;
-            case METH_SETTEXT:          MethSetText( pVar, pPar_, bWrite ); return;
+            case METH_SETDATA:          MethSetData( pPar_, bWrite ); return;
+            case METH_SETTEXT:          MethSetText( pPar_, bWrite ); return;
         }
 
         SbxObject::Notify( rBC, rHint );
diff --git a/basic/source/sbx/sbxdec.cxx b/basic/source/sbx/sbxdec.cxx
index d429ea9ca433..b0b7381a39eb 100644
--- a/basic/source/sbx/sbxdec.cxx
+++ b/basic/source/sbx/sbxdec.cxx
@@ -290,27 +290,23 @@ bool SbxDecimal::getDouble( double& rVal )
 #else
 // !WIN32
 
-bool SbxDecimal::operator -= ( const SbxDecimal &r )
+bool SbxDecimal::operator -= ( const SbxDecimal & )
 {
-    (void)r;
     return false;
 }
 
-bool SbxDecimal::operator += ( const SbxDecimal &r )
+bool SbxDecimal::operator += ( const SbxDecimal & )
 {
-    (void)r;
     return false;
 }
 
-bool SbxDecimal::operator /= ( const SbxDecimal &r )
+bool SbxDecimal::operator /= ( const SbxDecimal & )
 {
-    (void)r;
     return false;
 }
 
-bool SbxDecimal::operator *= ( const SbxDecimal &r )
+bool SbxDecimal::operator *= ( const SbxDecimal & )
 {
-    (void)r;
     return false;
 }
 
@@ -324,32 +320,30 @@ bool SbxDecimal::isZero()
     return false;
 }
 
-SbxDecimal::CmpResult compare( const SbxDecimal &rLeft, const SbxDecimal &rRight )
+SbxDecimal::CmpResult compare( SAL_UNUSED_PARAMETER const SbxDecimal &, SAL_UNUSED_PARAMETER const SbxDecimal & )
 {
-    (void)rLeft;
-    (void)rRight;
     return SbxDecimal::CmpResult::LT;
 }
 
-void SbxDecimal::setChar( sal_Unicode val )     { (void)val; }
-void SbxDecimal::setByte( sal_uInt8 val )           { (void)val; }
-void SbxDecimal::setShort( sal_Int16 val )          { (void)val; }
-void SbxDecimal::setLong( sal_Int32 val )           { (void)val; }
-void SbxDecimal::setUShort( sal_uInt16 val )        { (void)val; }
-void SbxDecimal::setULong( sal_uInt32 val )         { (void)val; }
-bool SbxDecimal::setSingle( float val )         { (void)val; return false; }
-bool SbxDecimal::setDouble( double val )        { (void)val; return false; }
-void SbxDecimal::setInt( int val )              { (void)val; }
-void SbxDecimal::setUInt( unsigned int val )    { (void)val; }
-bool SbxDecimal::setString( OUString* pOUString )    { (void)pOUString;  return false; }
-
-bool SbxDecimal::getChar( sal_Unicode& rVal )   { (void)rVal; return false; }
-bool SbxDecimal::getShort( sal_Int16& rVal )        { (void)rVal; return false; }
-bool SbxDecimal::getLong( sal_Int32& rVal )         { (void)rVal; return false; }
-bool SbxDecimal::getUShort( sal_uInt16& rVal )      { (void)rVal; return false; }
-bool SbxDecimal::getULong( sal_uInt32& rVal )       { (void)rVal; return false; }
-bool SbxDecimal::getSingle( float& rVal )       { (void)rVal; return false; }
-bool SbxDecimal::getDouble( double& rVal )      { (void)rVal; return false; }
+void SbxDecimal::setChar( SAL_UNUSED_PARAMETER sal_Unicode ) {}
+void SbxDecimal::setByte( SAL_UNUSED_PARAMETER sal_uInt8 ) {}
+void SbxDecimal::setShort( SAL_UNUSED_PARAMETER sal_Int16 ) {}
+void SbxDecimal::setLong( SAL_UNUSED_PARAMETER sal_Int32 ) {}
+void SbxDecimal::setUShort( SAL_UNUSED_PARAMETER sal_uInt16 ) {}
+void SbxDecimal::setULong( SAL_UNUSED_PARAMETER sal_uInt32 ) {}
+bool SbxDecimal::setSingle( SAL_UNUSED_PARAMETER float ) { return false; }
+bool SbxDecimal::setDouble( SAL_UNUSED_PARAMETER double ) { return false; }
+void SbxDecimal::setInt( SAL_UNUSED_PARAMETER int ) {}
+void SbxDecimal::setUInt( SAL_UNUSED_PARAMETER unsigned int ) {}
+bool SbxDecimal::setString( SAL_UNUSED_PARAMETER OUString* ) { return false; }
+
+bool SbxDecimal::getChar( SAL_UNUSED_PARAMETER sal_Unicode& ) { return false; }
+bool SbxDecimal::getShort( SAL_UNUSED_PARAMETER sal_Int16& ) { return false; }
+bool SbxDecimal::getLong( SAL_UNUSED_PARAMETER sal_Int32& ) { return false; }
+bool SbxDecimal::getUShort( SAL_UNUSED_PARAMETER sal_uInt16& ) { return false; }
+bool SbxDecimal::getULong( SAL_UNUSED_PARAMETER sal_uInt32& ) { return false; }
+bool SbxDecimal::getSingle( SAL_UNUSED_PARAMETER float& ) { return false; }
+bool SbxDecimal::getDouble( SAL_UNUSED_PARAMETER double& ) { return false; }
 
 #endif
 
diff --git a/basic/source/uno/scriptcont.cxx b/basic/source/uno/scriptcont.cxx
index 44ee7c787244..6f14015a0d9e 100644
--- a/basic/source/uno/scriptcont.cxx
+++ b/basic/source/uno/scriptcont.cxx
@@ -117,19 +117,17 @@ SfxScriptLibraryContainer::SfxScriptLibraryContainer( const uno::Reference< embe
 }
 
 // Methods to get library instances of the correct type
-SfxLibrary* SfxScriptLibraryContainer::implCreateLibrary( const OUString& aName )
+SfxLibrary* SfxScriptLibraryContainer::implCreateLibrary( const OUString& )
 {
-    (void)aName;    // Only needed for SfxDialogLibrary
     SfxLibrary* pRet = new SfxScriptLibrary( maModifiable, mxSFI );
     return pRet;
 }
 
-SfxLibrary* SfxScriptLibraryContainer::implCreateLibraryLink( const OUString& aName,
+SfxLibrary* SfxScriptLibraryContainer::implCreateLibraryLink( const OUString&,
                                                               const OUString& aLibInfoFileURL,
                                                               const OUString& StorageURL,
                                                               bool ReadOnly )
 {
-    (void)aName;    // Only needed for SfxDialogLibrary
     SfxLibrary* pRet = new SfxScriptLibrary( maModifiable, mxSFI,
                                              aLibInfoFileURL, StorageURL, ReadOnly );
     return pRet;
@@ -1172,25 +1170,18 @@ void SfxScriptLibrary::storeResources()
     // No resources
 }
 
-void SfxScriptLibrary::storeResourcesToURL( const OUString& URL,
-    const Reference< task::XInteractionHandler >& Handler )
-{
-    (void)URL;
-    (void)Handler;
-}
+void SfxScriptLibrary::storeResourcesToURL( const OUString&,
+    const Reference< task::XInteractionHandler >& )
+{}
 
 void SfxScriptLibrary::storeResourcesAsURL
-    ( const OUString& URL, const OUString& NewName )
-{
-    (void)URL;
-    (void)NewName;
-}
+    ( const OUString&, const OUString& )
+{}
 
 void SfxScriptLibrary::storeResourcesToStorage( const css::uno::Reference
-    < css::embed::XStorage >& xStorage )
+    < css::embed::XStorage >& )
 {
     // No resources
-    (void)xStorage;
 }
 
 bool SfxScriptLibrary::containsValidModule(const Any& rElement)