loplugin:expandablemethodds in scaddins..sdext

Change-Id: Ife021e368efaafe9097750b4ca1a5472e94352a9
Reviewed-on: https://gerrit.libreoffice.org/30054
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

scaddins/source/analysis/analysishelper.cxx

@@ -2182,7 +2182,7 @@ sal_Int16 ConvertData::GetMatchingLevel( const OUString& rRef ) const
         const sal_Unicode*  p = aStr.getStr();
 
         nLen = aStr.getLength();
-        bool bPref = IsPrefixSupport();
+        bool bPref = bPrefixSupport;
         bool bOneChar = (bPref && nLen > 1 && (aName == p + 1));
         if (bOneChar || (bPref && nLen > 2 && (aName == p + 2) &&
                     *p == 'd' && *(p+1) == 'a'))

scaddins/source/analysis/analysishelper.hxx

@@ -473,7 +473,6 @@ public:
     virtual double          ConvertFromBase( double fVal, sal_Int16 nMatchLevel ) const;
 
     inline ConvertDataClass Class() const;
-    inline bool         IsPrefixSupport() const;
 };
 
 class ConvertDataLinear : public ConvertData
@@ -659,11 +658,6 @@ inline ConvertDataClass ConvertData::Class() const
     return eClass;
 }
 
-inline bool ConvertData::IsPrefixSupport() const
-{
-    return bPrefixSupport;
-}
-
 inline ConvertDataLinear::ConvertDataLinear( const sal_Char* p, double fC, double fO, ConvertDataClass e,
         bool bPrefSupport ) :
     ConvertData( p, fC, e, bPrefSupport ),

scripting/source/provider/MasterScriptProvider.cxx

@@ -235,7 +235,7 @@ MasterScriptProvider::getScript( const OUString& scriptURI )
 throw ( provider::ScriptFrameworkErrorException,
         RuntimeException, std::exception )
 {
-    if ( !isValid() )
+    if ( !m_bIsValid )
     {
         throw provider::ScriptFrameworkErrorException(
             "MasterScriptProvider not initialised", Reference< XInterface >(),
@@ -389,7 +389,7 @@ OUString SAL_CALL
 MasterScriptProvider::getName()
         throw ( css::uno::RuntimeException, std::exception )
 {
-    if ( !isPkgProvider() )
+    if ( !m_bIsPkgMSP )
     {
         OUString sCtx = getContextString();
         if ( sCtx.startsWith( "vnd.sun.star.tdoc" ) )
@@ -421,9 +421,8 @@ MasterScriptProvider::getChildNodes()
 {
     Sequence< Reference< provider::XScriptProvider > > providers = getAllProviders();
 
-    Reference< provider::XScriptProvider > pkgProv = getPkgProvider();
     sal_Int32 size = providers.getLength();
-    bool hasPkgs = pkgProv.is();
+    bool hasPkgs = m_xMSPPkg.is();
     if ( hasPkgs  )
     {
         size++;
@@ -437,7 +436,7 @@ MasterScriptProvider::getChildNodes()
 
     if ( hasPkgs  )
     {
-        children[ provIndex ].set( pkgProv, UNO_QUERY );
+        children[ provIndex ].set( m_xMSPPkg, UNO_QUERY );
 
     }
 

scripting/source/provider/MasterScriptProvider.hxx

@@ -106,15 +106,12 @@ public:
     css::uno::Sequence< css::uno::Reference< css::script::provider::XScriptProvider > > SAL_CALL
         getAllProviders() throw ( css::uno::RuntimeException );
 
-    bool isPkgProvider() { return m_bIsPkgMSP; }
-    const css::uno::Reference< css::script::provider::XScriptProvider >& getPkgProvider() { return m_xMSPPkg; }
     // returns context string for this provider, eg
     const OUString& getContextString() { return m_sCtxString; }
 
 private:
     static OUString parseLocationName( const OUString& location );
     void  createPkgProvider();
-    bool  isValid() { return m_bIsValid;}
 
     ProviderCache* providerCache();
     /* to obtain other services if needed */

sd/inc/CustomAnimationPreset.hxx

@@ -92,8 +92,6 @@ public:
     SAL_DLLPRIVATE CustomAnimationPresets();
     SAL_DLLPRIVATE virtual ~CustomAnimationPresets();
 
-    SAL_DLLPRIVATE void init();
-
     static const CustomAnimationPresets& getCustomAnimationPresets();
 
     SAL_DLLPRIVATE css::uno::Reference< css::animations::XAnimationNode > getRandomPreset( sal_Int16 nPresetClass ) const;

sd/inc/cusshow.hxx

@@ -65,8 +65,6 @@ public:
     void   SetName(const OUString& rName);
     const OUString& GetName() const { return aName;}
 
-    SdDrawDocument* GetDoc() const { return pDoc; }
-
     css::uno::Reference< css::uno::XInterface > getUnoCustomShow();
 };
 

sd/inc/sdiocmpt.hxx

@@ -32,7 +32,6 @@ protected:
     bool                        bOpen;
 
 protected:
-    void Read();
     void Write();
 
 public:

sd/source/core/CustomAnimationPreset.cxx

@@ -234,11 +234,6 @@ CustomAnimationPresets::~CustomAnimationPresets()
 {
 }
 
-void CustomAnimationPresets::init()
-{
-    importResources();
-}
-
 Reference< XAnimationNode > implImportEffects( const Reference< XMultiServiceFactory >& xServiceFactory, const OUString& rPath )
 {
     Reference< XAnimationNode > xRootNode;
@@ -552,7 +547,7 @@ const CustomAnimationPresets& CustomAnimationPresets::getCustomAnimationPresets(
         if( !mpCustomAnimationPresets )
         {
             mpCustomAnimationPresets = new sd::CustomAnimationPresets();
-            mpCustomAnimationPresets->init();
+            mpCustomAnimationPresets->importResources();
         }
     }
 

sd/source/core/cusshow.cxx

@@ -49,7 +49,7 @@ SdCustomShow::SdCustomShow( const SdCustomShow& rShow )
     : maPages(rShow.maPages)
 {
     aName = rShow.GetName();
-    pDoc = rShow.GetDoc();
+    pDoc = rShow.pDoc;
 }
 
 SdCustomShow::SdCustomShow(SdDrawDocument* pDrawDoc, css::uno::Reference< css::uno::XInterface > const & xShow )

sd/source/core/sdiocmpt.cxx

@@ -37,11 +37,6 @@ old_SdrDownCompat::~old_SdrDownCompat()
         CloseSubRecord();
 }
 
-void old_SdrDownCompat::Read()
-{
-    rStream.ReadUInt32( nSubRecSiz );
-}
-
 void old_SdrDownCompat::Write()
 {
     rStream.WriteUInt32( nSubRecSiz );
@@ -56,7 +51,7 @@ void old_SdrDownCompat::OpenSubRecord()
 
     if(nMode == StreamMode::READ)
     {
-        Read();
+        rStream.ReadUInt32( nSubRecSiz );
     }
     else if(nMode == StreamMode::WRITE)
     {

sd/source/ui/dlg/headerfooterdlg.cxx

@@ -274,7 +274,7 @@ IMPL_LINK_NOARG(HeaderFooterDialog, ClickApplyHdl, Button*, void)
 
 IMPL_LINK_NOARG(HeaderFooterDialog, ClickCancelHdl, Button*, void)
 {
-    Cancel();
+    EndDialog();
 }
 
 short HeaderFooterDialog::Execute()
@@ -299,11 +299,6 @@ void HeaderFooterDialog::Apply()
     EndDialog(1);
 }
 
-void HeaderFooterDialog::Cancel()
-{
-    EndDialog();
-}
-
 void HeaderFooterDialog::apply( bool bToAll, bool bForceSlides )
 {
     SdUndoGroup* pUndoGroup = new SdUndoGroup(mpDoc);

sd/source/ui/inc/PreviewRenderer.hxx

@@ -127,7 +127,6 @@ private:
         const SdPage* pPage,
         const Size& rPixelSize,
         const bool bObeyHighContrastMode);
-    void Cleanup();
     void PaintPage (
         const SdPage* pPage,
         const bool bDisplayPresentationObjects);

sd/source/ui/inc/headerfooterdlg.hxx

@@ -71,7 +71,6 @@ public:
 
     void ApplyToAll();
     void Apply();
-    void Cancel();
 
     virtual short Execute() override;
 };

sd/source/ui/tools/PreviewRenderer.cxx

@@ -127,7 +127,7 @@ Image PreviewRenderer::RenderPage (
                     mpPreviewDevice->PixelToLogic(Point(0,0)),
                     mpPreviewDevice->PixelToLogic(aSize)));
 
-                Cleanup();
+                mpView->HideSdrPage();
             }
         }
         catch (const css::uno::Exception&)
@@ -272,11 +272,6 @@ bool PreviewRenderer::Initialize (
     return true;
 }
 
-void PreviewRenderer::Cleanup()
-{
-    mpView->HideSdrPage();
-}
-
 void PreviewRenderer::PaintPage (
     const SdPage* pPage,
     const bool bDisplayPresentationObjects)

sd/source/ui/view/DocumentRenderer.cxx

@@ -1182,7 +1182,7 @@ public:
 
         if (rHint.GetId() == SFX_HINT_DYING)
         {
-            Dispose();
+            mbIsDisposed = true;
         }
     }
 
@@ -1352,11 +1352,6 @@ private:
     std::vector<sal_Int32> maSlidesPerPage;
     awt::Size maPrintSize;
 
-    void Dispose()
-    {
-        mbIsDisposed = true;
-    }
-
     sal_Int32 GetCurrentPageIndex() const
     {
         const ViewShell *pShell = mrBase.GetMainViewShell().get();

sdext/source/presenter/PresenterAccessibility.cxx

@@ -496,7 +496,6 @@ private:
         const sal_Int32 nOldCharacterIndex,
         const sal_Int32 nNewParagraphIndex,
         const sal_Int32 nNewCharacterIndex);
-    void HandleTextChange();
 };
 
 //===== AccessibleFocusManager ================================================
@@ -1912,7 +1911,7 @@ void AccessibleNotes::SetTextView (
             [this](sal_Int32 a, sal_Int32 b, sal_Int32 c, sal_Int32 d)
                 { return this->NotifyCaretChange(a, b, c, d); });
         mpTextView->SetTextChangeBroadcaster(
-            [this]() { return this->HandleTextChange(); });
+            [this]() { return SetTextView(mpTextView); });
     }
 }
 
@@ -1975,10 +1974,6 @@ void AccessibleNotes::NotifyCaretChange (
     }
 }
 
-void AccessibleNotes::HandleTextChange()
-{
-    SetTextView(mpTextView);
-}
 
 //===== AccessibleFocusManager ================================================