loplugin:constantparam in framework..vbahelper

Change-Id: I592f73012ae8a4df8488ef04dc0956d9400625bc

3 files changed, 9 insertions, 14 deletions

diff --git a/unotools/source/config/cmdoptions.cxx b/unotools/source/config/cmdoptions.cxx
index 0f53f1ea52aa..50f816c19ab9 100644
--- a/unotools/source/config/cmdoptions.cxx
+++ b/unotools/source/config/cmdoptions.cxx
@@ -115,7 +115,7 @@ class SvtCommandOptions_Impl : public ConfigItem
 
         bool                HasEntries  (   SvtCommandOptions::CmdOption    eOption     ) const;
         bool                Lookup      (   SvtCommandOptions::CmdOption    eCmdOption, const OUString& ) const;
-        void EstablisFrameCallback(const css::uno::Reference< css::frame::XFrame >& xFrame);
+        void EstablishFrameCallback(const css::uno::Reference< css::frame::XFrame >& xFrame);
 
     private:
 
@@ -260,7 +260,7 @@ bool SvtCommandOptions_Impl::Lookup( SvtCommandOptions::CmdOption eCmdOption, co
 
 //  public method
 
-void SvtCommandOptions_Impl::EstablisFrameCallback(const css::uno::Reference< css::frame::XFrame >& xFrame)
+void SvtCommandOptions_Impl::EstablishFrameCallback(const css::uno::Reference< css::frame::XFrame >& xFrame)
 {
     // check if frame already exists inside list
     // ignore double registrations
@@ -335,10 +335,10 @@ bool SvtCommandOptions::Lookup( CmdOption eCmdOption, const OUString& aCommandUR
 
 //  public method
 
-void SvtCommandOptions::EstablisFrameCallback(const css::uno::Reference< css::frame::XFrame >& xFrame)
+void SvtCommandOptions::EstablishFrameCallback(const css::uno::Reference< css::frame::XFrame >& xFrame)
 {
     MutexGuard aGuard( GetOwnStaticMutex() );
-    m_pImpl->EstablisFrameCallback(xFrame);
+    m_pImpl->EstablishFrameCallback(xFrame);
 }
 
 namespace
diff --git a/unotools/source/config/configvaluecontainer.cxx b/unotools/source/config/configvaluecontainer.cxx
index df0a002d4753..873dcfef85d7 100644
--- a/unotools/source/config/configvaluecontainer.cxx
+++ b/unotools/source/config/configvaluecontainer.cxx
@@ -281,11 +281,10 @@ namespace utl
         );
     }
 
-    void OConfigurationValueContainer::commit( bool _bWrite )
+    void OConfigurationValueContainer::commit()
     {
-        // write the current values in the exchange locations (if requested)
-        if ( _bWrite )
-            write();
+        // write the current values in the exchange locations
+        write();
 
         // commit the changes done
         m_pImpl->aConfigRoot.commit( );
diff --git a/unotools/source/i18n/transliterationwrapper.cxx b/unotools/source/i18n/transliterationwrapper.cxx
index a62e54ef3f9f..4e69bb9eaa02 100644
--- a/unotools/source/i18n/transliterationwrapper.cxx
+++ b/unotools/source/i18n/transliterationwrapper.cxx
@@ -68,18 +68,14 @@ OUString TransliterationWrapper::transliterate(const OUString& rStr, sal_uInt16
 }
 
 OUString TransliterationWrapper::transliterate( const OUString& rStr,
-                                                sal_Int32 nStart, sal_Int32 nLen,
-                                                Sequence <sal_Int32>* pOffset ) const
+                                                sal_Int32 nStart, sal_Int32 nLen ) const
 {
     OUString sRet( rStr );
     if( xTrans.is() )
     {
         try
         {
-            if ( pOffset )
-                sRet = xTrans->transliterate( rStr, nStart, nLen, *pOffset );
-            else
-                sRet = xTrans->transliterateString2String( rStr, nStart, nLen);
+            sRet = xTrans->transliterateString2String( rStr, nStart, nLen);
         }
         catch( Exception&  )
         {