[Libreoffice-commits] core.git: extensions/source include/svtools svtools/source

Noel Grandin noel.grandin at collabora.co.uk
Thu Oct 27 10:23:58 UTC 2016


 extensions/source/propctrlr/taborder.hxx    |    3 -
 include/svtools/PlaceEditDialog.hxx         |    1 
 include/svtools/accessibleruler.hxx         |    8 ----
 include/svtools/brwbox.hxx                  |    2 -
 include/svtools/calendar.hxx                |    1 
 include/svtools/dialogcontrolling.hxx       |    1 
 include/svtools/filectrl.hxx                |    3 -
 include/svtools/grfmgr.hxx                  |    5 --
 include/svtools/simptabl.hxx                |    1 
 include/svtools/treelistbox.hxx             |    3 -
 include/svtools/unoevent.hxx                |    1 
 include/svtools/wizdlg.hxx                  |    1 
 svtools/source/brwbox/brwbox2.cxx           |    2 -
 svtools/source/config/extcolorcfg.cxx       |    8 ----
 svtools/source/contnr/fileview.cxx          |   24 +++++--------
 svtools/source/contnr/imivctl.hxx           |    3 -
 svtools/source/contnr/imivctl1.cxx          |   11 +-----
 svtools/source/contnr/simptabl.cxx          |   20 ++++-------
 svtools/source/contnr/svimpbox.cxx          |    7 ----
 svtools/source/contnr/treelistbox.cxx       |   22 +-----------
 svtools/source/control/accessibleruler.cxx  |    2 -
 svtools/source/control/calendar.cxx         |    7 ----
 svtools/source/control/filectrl.cxx         |    7 ----
 svtools/source/control/toolbarmenu.cxx      |    8 ----
 svtools/source/control/toolbarmenuacc.cxx   |   19 +++-------
 svtools/source/control/toolbarmenuimp.hxx   |    3 -
 svtools/source/dialogs/PlaceEditDialog.cxx  |   49 ++++++++++++----------------
 svtools/source/dialogs/addresstemplate.cxx  |    9 -----
 svtools/source/dialogs/wizdlg.cxx           |    7 ----
 svtools/source/graphic/descriptor.cxx       |   19 +---------
 svtools/source/graphic/descriptor.hxx       |    3 -
 svtools/source/graphic/graphic.cxx          |   17 +--------
 svtools/source/graphic/graphic.hxx          |    2 -
 svtools/source/graphic/grfmgr.cxx           |   18 ++++------
 svtools/source/inc/svimpbox.hxx             |    3 -
 svtools/source/inc/unoiface.hxx             |    3 -
 svtools/source/misc/dialogcontrolling.cxx   |   11 +-----
 svtools/source/misc/templatefoldercache.cxx |    9 -----
 svtools/source/table/tabledatawindow.cxx    |    5 --
 svtools/source/table/tabledatawindow.hxx    |    2 -
 svtools/source/uno/unoevent.cxx             |   13 +------
 svtools/source/uno/unoiface.cxx             |    2 -
 42 files changed, 80 insertions(+), 265 deletions(-)

New commits:
commit 7da765dde1800aa607f96e7a5807582ad2894acb
Author: Noel Grandin <noel.grandin at collabora.co.uk>
Date:   Thu Oct 27 10:21:42 2016 +0200

    loplugin:expandablemethods in svtools
    
    Change-Id: I126cc87c890867c3e762f86969a20d64aef63108
    Reviewed-on: https://gerrit.libreoffice.org/30314
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/extensions/source/propctrlr/taborder.hxx b/extensions/source/propctrlr/taborder.hxx
index 4e35156..6569086 100644
--- a/extensions/source/propctrlr/taborder.hxx
+++ b/extensions/source/propctrlr/taborder.hxx
@@ -52,9 +52,6 @@ namespace pcr
 
     protected:
         virtual void    ModelHasMoved(SvTreeListEntry* pSource ) override;
-
-    private:
-        using SvTreeListBox::MoveSelection;
     };
 
 
diff --git a/include/svtools/PlaceEditDialog.hxx b/include/svtools/PlaceEditDialog.hxx
index c6ab42b..4649307 100644
--- a/include/svtools/PlaceEditDialog.hxx
+++ b/include/svtools/PlaceEditDialog.hxx
@@ -81,7 +81,6 @@ public :
 private:
 
     void InitDetails( );
-    void UpdateLabel( );
 
     DECL_LINK ( OKHdl, Button *, void );
     DECL_LINK ( DelHdl, Button *, void );
diff --git a/include/svtools/accessibleruler.hxx b/include/svtools/accessibleruler.hxx
index ddc63a4..2d4861f 100644
--- a/include/svtools/accessibleruler.hxx
+++ b/include/svtools/accessibleruler.hxx
@@ -178,9 +178,6 @@ protected:
     /// @returns true if it's disposed or in disposing
     inline bool IsAlive() const;
 
-    /// @returns true if it's not disposed and no in disposing
-    inline bool IsNotAlive() const;
-
     /// throws the exception DisposedException if it's not alive
     void ThrowExceptionIfNotAlive() throw( css::lang::DisposedException );
 
@@ -212,11 +209,6 @@ inline bool SvtRulerAccessible::IsAlive() const
     return !rBHelper.bDisposed && !rBHelper.bInDispose;
 }
 
-inline bool SvtRulerAccessible::IsNotAlive() const
-{
-    return rBHelper.bDisposed || rBHelper.bInDispose;
-}
-
 #endif
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/include/svtools/brwbox.hxx b/include/svtools/brwbox.hxx
index 5ec84e6..903fce8 100644
--- a/include/svtools/brwbox.hxx
+++ b/include/svtools/brwbox.hxx
@@ -302,8 +302,6 @@ private:
 
     bool            GoToColumnId( sal_uInt16 nColId, bool bMakeVisible, bool bRowColMove = false);
     void            SelectColumnPos( sal_uInt16 nCol, bool _bSelect, bool bMakeVisible);
-    void            SelectColumnId( sal_uInt16 nColId, bool _bSelect, bool bMakeVisible)
-                        { SelectColumnPos( GetColumnPos(nColId), _bSelect, bMakeVisible); }
 
     void            ImplPaintData(OutputDevice& _rOut, const Rectangle& _rRect, bool _bForeignDevice, bool _bDrawSelections);
 
diff --git a/include/svtools/calendar.hxx b/include/svtools/calendar.hxx
index 606d0dd..8cd906d 100644
--- a/include/svtools/calendar.hxx
+++ b/include/svtools/calendar.hxx
@@ -354,7 +354,6 @@ public:
     virtual void        dispose() override;
 
     virtual bool        ShowDropDown( bool bShow ) override;
-    VclPtr<Calendar>    CreateCalendar( vcl::Window* pParent );
     Calendar*           GetCalendar();
 
     void                EnableToday() { mbToday = true; }
diff --git a/include/svtools/dialogcontrolling.hxx b/include/svtools/dialogcontrolling.hxx
index e934adf..020572b 100644
--- a/include/svtools/dialogcontrolling.hxx
+++ b/include/svtools/dialogcontrolling.hxx
@@ -111,7 +111,6 @@ namespace svt
         void    reset();
 
     private:
-        void    impl_updateAll( const VclWindowEvent& _rTriggerEvent );
         void    impl_update( const VclWindowEvent& _rTriggerEvent, vcl::Window& _rWindow );
 
         DECL_LINK( OnWindowEvent, VclWindowEvent&, void );
diff --git a/include/svtools/filectrl.hxx b/include/svtools/filectrl.hxx
index 7108f44..1dbb4f3 100644
--- a/include/svtools/filectrl.hxx
+++ b/include/svtools/filectrl.hxx
@@ -50,9 +50,6 @@ private:
 
     FileControlMode_Internal    mnInternalFlags;
 
-private:
-    SVT_DLLPRIVATE void     ImplBrowseFile( );
-
 protected:
     SVT_DLLPRIVATE void     Resize() override;
     SVT_DLLPRIVATE void     GetFocus() override;
diff --git a/include/svtools/grfmgr.hxx b/include/svtools/grfmgr.hxx
index 66cb2ff..75fb774 100644
--- a/include/svtools/grfmgr.hxx
+++ b/include/svtools/grfmgr.hxx
@@ -311,9 +311,6 @@ private:
 
                             DECL_LINK( ImplAutoSwapOutHdl, Timer*, void );
 
-    // restart SwapOut timer; this is like touching in a cache to reset to the full timeout value
-    void SVT_DLLPRIVATE     restartSwapOutTimer() const;
-
     // Handle evtl. needed AfterDataChanges, needs to be called when new
     // graphic data is swapped in/added to the GraphicManager
     void SVT_DLLPRIVATE     ImplAfterDataChange();
@@ -409,8 +406,6 @@ public:
     bool                    IsAnimated() const { return mbAnimated; }
     bool                    IsEPS() const { return mbEPS; }
 
-    Link<Animation*,void>   GetAnimationNotifyHdl() const { return GetGraphic().GetAnimationNotifyHdl(); }
-
     bool                    SwapOut();
     bool                    SwapOut( SvStream* pOStm );
     bool                    SwapIn();
diff --git a/include/svtools/simptabl.hxx b/include/svtools/simptabl.hxx
index 3890e44..5f180ea 100644
--- a/include/svtools/simptabl.hxx
+++ b/include/svtools/simptabl.hxx
@@ -80,7 +80,6 @@ protected:
     virtual void            Paint( vcl::RenderContext& rRenderContext, const Rectangle& rRect ) override;
 
     virtual void            HBarClick();
-    void                    HBarStartDrag();
     void                    HBarDrag();
     void                    HBarEndDrag();
 
diff --git a/include/svtools/treelistbox.hxx b/include/svtools/treelistbox.hxx
index 093d520..d9ac1b5 100644
--- a/include/svtools/treelistbox.hxx
+++ b/include/svtools/treelistbox.hxx
@@ -305,7 +305,6 @@ protected:
     void            EditText( const OUString&, const Rectangle&,const Selection&);
     void            CancelTextEditing();
     bool            EditingCanceled() const;
-    bool            IsEmptyTextAllowed() const;
 
     // Return value must be derived from SvViewDataEntry!
     virtual SvViewDataEntry* CreateViewData( SvTreeListEntry* ) override;
@@ -380,7 +379,6 @@ public:
     static SvTreeListEntry* PrevSibling( SvTreeListEntry* pEntry );
 
     bool            CopySelection( SvTreeListBox* pSource, SvTreeListEntry* pTarget );
-    bool            MoveSelection( SvTreeListBox* pSource, SvTreeListEntry* pTarget );
     bool            MoveSelectionCopyFallbackPossible( SvTreeListBox* pSource, SvTreeListEntry* pTarget, bool bAllowCopyFallback );
     void            RemoveSelection();
 
@@ -535,7 +533,6 @@ protected:
     using SvListView::SelectAll;
 
     SVT_DLLPRIVATE short        GetHeightOffset( const Image& rBmp, Size& rLogicSize);
-    SVT_DLLPRIVATE void         GetHeightOffset( Size& rLogicSize);
 
     SVT_DLLPRIVATE void         SetEntryHeight( SvTreeListEntry* pEntry );
     SVT_DLLPRIVATE void         AdjustEntryHeight( const Image& rBmp );
diff --git a/include/svtools/unoevent.hxx b/include/svtools/unoevent.hxx
index ef30363..e5332cb 100644
--- a/include/svtools/unoevent.hxx
+++ b/include/svtools/unoevent.hxx
@@ -280,7 +280,6 @@ public:
 
     virtual ~SvMacroTableEventDescriptor() override;
 
-    void copyMacrosFromTable(const SvxMacroTableDtor& aFmt);
     void copyMacrosIntoTable(SvxMacroTableDtor& aFmt);
 };
 
diff --git a/include/svtools/wizdlg.hxx b/include/svtools/wizdlg.hxx
index 91473c4..a115a1a 100644
--- a/include/svtools/wizdlg.hxx
+++ b/include/svtools/wizdlg.hxx
@@ -204,7 +204,6 @@ private:
     bool                    mbEmptyViewMargin;
 
     DECL_DLLPRIVATE_LINK( ImplHandleWizardLayoutTimerHdl, Idle*, void );
-    bool hasWizardPendingLayout() const;
 
 protected:
     long                LogicalCoordinateToPixel(int iCoordinate);
diff --git a/svtools/source/brwbox/brwbox2.cxx b/svtools/source/brwbox/brwbox2.cxx
index cfc4ab9..e1f12bc 100644
--- a/svtools/source/brwbox/brwbox2.cxx
+++ b/svtools/source/brwbox/brwbox2.cxx
@@ -1600,7 +1600,7 @@ void BrowseBox::MouseButtonDown( const BrowserMouseEvent& rEvt )
                         SelectAll();
                 }
                 else
-                    SelectColumnId( rEvt.GetColumnId(), true, false );
+                    SelectColumnPos( GetColumnPos(rEvt.GetColumnId()), true, false);
             }
 
             // turn cursor on again, if necessary
diff --git a/svtools/source/config/extcolorcfg.cxx b/svtools/source/config/extcolorcfg.cxx
index 96e61e5..3fd7601 100644
--- a/svtools/source/config/extcolorcfg.cxx
+++ b/svtools/source/config/extcolorcfg.cxx
@@ -125,7 +125,6 @@ public:
 
     static void                     DisableBroadcast();
     static void                     EnableBroadcast();
-    static bool                     IsEnableBroadcast();
 
     static void                     LockBroadcast();
     static void                     UnlockBroadcast();
@@ -224,11 +223,6 @@ void ExtendedColorConfig_Impl::EnableBroadcast()
         ExtendedColorConfig::m_pImpl->m_bIsBroadcastEnabled = true;
 }
 
-bool ExtendedColorConfig_Impl::IsEnableBroadcast()
-{
-    return ExtendedColorConfig::m_pImpl && ExtendedColorConfig::m_pImpl->m_bIsBroadcastEnabled;
-}
-
 void lcl_addString(uno::Sequence < OUString >& _rSeq,const OUString& _sAdd)
 {
     OUString* pIter = _rSeq.getArray();
@@ -516,7 +510,7 @@ void ExtendedColorConfig_Impl::UnlockBroadcast()
         m_bBroadcastWhenUnlocked = ExtendedColorConfig::m_pImpl != nullptr;
         if ( m_bBroadcastWhenUnlocked )
         {
-            if ( ExtendedColorConfig_Impl::IsEnableBroadcast() )
+            if ( ExtendedColorConfig::m_pImpl && ExtendedColorConfig::m_pImpl->m_bIsBroadcastEnabled )
             {
                 m_bBroadcastWhenUnlocked = false;
                 ExtendedColorConfig::m_pImpl->Broadcast(SfxHint(SFX_HINT_COLORS_CHANGED));
diff --git a/svtools/source/contnr/fileview.cxx b/svtools/source/contnr/fileview.cxx
index 6343ae7..76f1ea5 100644
--- a/svtools/source/contnr/fileview.cxx
+++ b/svtools/source/contnr/fileview.cxx
@@ -219,7 +219,6 @@ protected:
 private:
     std::unordered_map<OUString, OUString, OUStringHash> m_Translation;
     const OUString              maTransFileName;
-    void                        Init();         // reads the translation file and fills the (internal) list
 
 public:
                                 explicit NameTranslationList( const INetURLObject& rBaseURL );
@@ -239,13 +238,17 @@ inline const OUString& NameTranslationList::GetTransTableFileName() const
     return maTransFileName;
 }
 
-void NameTranslationList::Init()
+NameTranslationList::NameTranslationList( const INetURLObject& rBaseURL ):
+    maTransFile( rBaseURL ),
+    m_HashedURL(rBaseURL.GetMainURL(INetURLObject::NO_DECODE)),
+    maTransFileName( OUString(".nametranslation.table") )
 {
-// Tries to read the file ".nametranslation.table" in the base folder. Complete path/name is in maTransFile.
-// Further on, the found entries in the section "TRANSLATIONNAMES" are used to replace names in the
-// base folder by translated ones. The translation must be given in UTF8
-// See examples of such a files in the samples-folder of an Office installation
+    maTransFile.insertName( maTransFileName );
 
+    // Tries to read the file ".nametranslation.table" in the base folder. Complete path/name is in maTransFile.
+    // Further on, the found entries in the section "TRANSLATIONNAMES" are used to replace names in the
+    // base folder by translated ones. The translation must be given in UTF8
+    // See examples of such a files in the samples-folder of an Office installation
     try
     {
         ::ucbhelper::Content aTestContent( maTransFile.GetMainURL( INetURLObject::NO_DECODE ), Reference< XCommandEnvironment >(), comphelper::getProcessComponentContext() );
@@ -272,15 +275,6 @@ void NameTranslationList::Init()
     catch( Exception const & ) {}
 }
 
-NameTranslationList::NameTranslationList( const INetURLObject& rBaseURL ):
-    maTransFile( rBaseURL ),
-    m_HashedURL(rBaseURL.GetMainURL(INetURLObject::NO_DECODE)),
-    maTransFileName( OUString(".nametranslation.table") )
-{
-    maTransFile.insertName( maTransFileName );
-    Init();
-}
-
 const OUString* NameTranslationList::Translate( const OUString& rName ) const
 {
     auto const iter(m_Translation.find(rName));
diff --git a/svtools/source/contnr/imivctl.hxx b/svtools/source/contnr/imivctl.hxx
index 4bac273..85ea19a 100644
--- a/svtools/source/contnr/imivctl.hxx
+++ b/svtools/source/contnr/imivctl.hxx
@@ -247,7 +247,6 @@ class SvxIconChoiceCtrl_Impl
     void                Center( SvxIconChoiceCtrlEntry* pEntry ) const;
     void                StopEditTimer() { aEditIdle.Stop(); }
     void                StartEditTimer() { aEditIdle.Start(); }
-    void                ImpHideDDIcon();
     void                CallSelectHandler( SvxIconChoiceCtrlEntry* );
     void                SelectRect(
                             SvxIconChoiceCtrlEntry* pEntry1,
@@ -478,8 +477,6 @@ public:
     void                SetColumn( sal_uInt16 nIndex, const SvxIconChoiceCtrlColumnInfo& );
     const SvxIconChoiceCtrlColumnInfo* GetColumn( sal_uInt16 nIndex ) const;
 
-    Rectangle           GetDocumentRect() const { return Rectangle( Point(), aVirtOutputSize ); }
-
     void                SetEntryHighlightFrame(
                             SvxIconChoiceCtrlEntry* pEntry,
                             bool bKeepHighlightFlags
diff --git a/svtools/source/contnr/imivctl1.cxx b/svtools/source/contnr/imivctl1.cxx
index 82b8af5..a07194f 100644
--- a/svtools/source/contnr/imivctl1.cxx
+++ b/svtools/source/contnr/imivctl1.cxx
@@ -2016,24 +2016,19 @@ void SvxIconChoiceCtrl_Impl::ShowCursor( bool bShow )
 void SvxIconChoiceCtrl_Impl::HideDDIcon()
 {
     pView->Update();
-    ImpHideDDIcon();
-    pDDBufDev = pDDDev;
-    pDDDev = nullptr;
-}
-
-void SvxIconChoiceCtrl_Impl::ImpHideDDIcon()
-{
     if( pDDDev )
     {
         Size aSize( pDDDev->GetOutputSizePixel() );
         // restore pView
         pView->DrawOutDev( aDDLastRectPos, aSize, Point(), aSize, *pDDDev );
     }
+    pDDBufDev = pDDDev;
+    pDDDev = nullptr;
 }
 
 bool SvxIconChoiceCtrl_Impl::HandleScrollCommand( const CommandEvent& rCmd )
 {
-    Rectangle aDocRect( GetDocumentRect() );
+    Rectangle aDocRect( Point(), aVirtOutputSize );
     Rectangle aVisRect( GetOutputRect() );
     if( aVisRect.IsInside( aDocRect ))
         return false;
diff --git a/svtools/source/contnr/simptabl.cxx b/svtools/source/contnr/simptabl.cxx
index 0652388..966596f 100644
--- a/svtools/source/contnr/simptabl.cxx
+++ b/svtools/source/contnr/simptabl.cxx
@@ -344,17 +344,6 @@ void SvSimpleTable::HBarClick()
     }
 }
 
-void SvSimpleTable::HBarStartDrag()
-{
-    if(!aHeaderBar->IsItemMode())
-    {
-        Rectangle aSizeRect(Point(0,0),
-            SvHeaderTabListBox::GetOutputSizePixel());
-        aSizeRect.Left()=-GetXOffset()+aHeaderBar->GetDragPos();
-        aSizeRect.Right()=-GetXOffset()+aHeaderBar->GetDragPos();
-        ShowTracking( aSizeRect, ShowTrackFlags::Split );
-    }
-}
 void SvSimpleTable::HBarDrag()
 {
     HideTracking();
@@ -403,7 +392,14 @@ IMPL_LINK( SvSimpleTable, StartDragHdl, HeaderBar*, pCtr, void)
 {
     if(pCtr==aHeaderBar.get())
     {
-        HBarStartDrag();
+        if(!aHeaderBar->IsItemMode())
+        {
+            Rectangle aSizeRect(Point(0,0),
+                SvHeaderTabListBox::GetOutputSizePixel());
+            aSizeRect.Left()=-GetXOffset()+aHeaderBar->GetDragPos();
+            aSizeRect.Right()=-GetXOffset()+aHeaderBar->GetDragPos();
+            ShowTracking( aSizeRect, ShowTrackFlags::Split );
+        }
     }
 }
 
diff --git a/svtools/source/contnr/svimpbox.cxx b/svtools/source/contnr/svimpbox.cxx
index 605d227..6cd671b 100644
--- a/svtools/source/contnr/svimpbox.cxx
+++ b/svtools/source/contnr/svimpbox.cxx
@@ -2194,7 +2194,7 @@ bool SvImpLBox::KeyInput( const KeyEvent& rKEvt)
             {
                 // only try to expand if sublist is expandable,
                 // otherwise ignore the key press
-                if( IsNowExpandable() )
+                if( IsExpandable() && !pView->IsExpanded( pCursor ) )
                     pView->Expand( pCursor );
             }
             else if ( bIsCellFocusEnabled && pCursor )
@@ -3306,11 +3306,6 @@ bool SvImpLBox::IsExpandable() const
     return pCursor->HasChildren() || pCursor->HasChildrenOnDemand();
 }
 
-bool SvImpLBox::IsNowExpandable() const
-{
-    return IsExpandable() && !pView->IsExpanded( pCursor );
-}
-
 IMPL_LINK(SvImpLBox, MyUserEvent, void*, pArg, void )
 {
     nCurUserEvent = nullptr;
diff --git a/svtools/source/contnr/treelistbox.cxx b/svtools/source/contnr/treelistbox.cxx
index d025ee6..6ab4ab7 100644
--- a/svtools/source/contnr/treelistbox.cxx
+++ b/svtools/source/contnr/treelistbox.cxx
@@ -602,11 +602,6 @@ bool SvTreeListBox::CopySelection( SvTreeListBox* pSource, SvTreeListEntry* pTar
 }
 
 // return: all entries were moved
-bool SvTreeListBox::MoveSelection( SvTreeListBox* pSource, SvTreeListEntry* pTarget )
-{
-    return MoveSelectionCopyFallbackPossible( pSource, pTarget, false );
-}
-
 bool SvTreeListBox::MoveSelectionCopyFallbackPossible( SvTreeListBox* pSource, SvTreeListEntry* pTarget, bool bAllowCopyFallback )
 {
     nCurEntrySelPos = 0; // selection counter for NotifyMoving/Copying
@@ -917,7 +912,7 @@ IMPL_LINK_NOARG(SvTreeListBox, TextEditEndedHdl_Impl, SvInplaceEdit2&, void)
         aStr = pEdCtrl->GetText();
     else
         aStr = pEdCtrl->GetSavedValue();
-    if ( IsEmptyTextAllowed() || !aStr.isEmpty() )
+    if ( mpImpl->m_bIsEmptyTextAllowed || !aStr.isEmpty() )
         EditedText( aStr );
     // Hide may only be called after the new text was put into the entry, so
     // that we don't call the selection handler in the GetFocus of the listbox
@@ -942,11 +937,6 @@ void SvTreeListBox::EndEditing( bool bCancel )
 }
 
 
-bool SvTreeListBox::IsEmptyTextAllowed() const
-{
-    return mpImpl->m_bIsEmptyTextAllowed;
-}
-
 void SvTreeListBox::ForbidEmptyText()
 {
     mpImpl->m_bIsEmptyTextAllowed = false;
@@ -1141,7 +1131,7 @@ sal_Int8 SvTreeListBox::ExecuteDrop( const ExecuteDropEvent& rEvt, SvTreeListBox
         }
         else if( DND_ACTION_MOVE == rEvt.mnAction )
         {
-            if (MoveSelection(g_pDDSource, pTarget))
+            if (MoveSelectionCopyFallbackPossible( g_pDDSource, pTarget, false ))
                 nRet = rEvt.mnAction;
         }
         else if( DND_ACTION_COPYMOVE == rEvt.mnAction )
@@ -2156,11 +2146,6 @@ short SvTreeListBox::GetHeightOffset(const Image& rBmp, Size& aSizeLogic )
     return nOffset;
 }
 
-void SvTreeListBox::GetHeightOffset(Size& aSizeLogic )
-{
-    aSizeLogic = Size(GetTextWidth(OUString('X')), GetTextHeight());
-}
-
 void SvTreeListBox::SetEntryHeight( SvTreeListEntry* pEntry )
 {
     short nHeightMax=0;
@@ -2215,8 +2200,7 @@ void SvTreeListBox::AdjustEntryHeight( const Image& rBmp )
 
 void SvTreeListBox::AdjustEntryHeight()
 {
-    Size aSize;
-    GetHeightOffset( aSize );
+    Size aSize( GetTextWidth(OUString('X')), GetTextHeight() );
     if( aSize.Height()  >  nEntryHeight )
     {
         nEntryHeight = (short)aSize.Height() + nEntryHeightOffs;
diff --git a/svtools/source/control/accessibleruler.cxx b/svtools/source/control/accessibleruler.cxx
index 3753d45..bd71793 100644
--- a/svtools/source/control/accessibleruler.cxx
+++ b/svtools/source/control/accessibleruler.cxx
@@ -373,7 +373,7 @@ Rectangle SvtRulerAccessible::GetBoundingBox() throw( RuntimeException )
 
 void SvtRulerAccessible::ThrowExceptionIfNotAlive() throw( lang::DisposedException )
 {
-    if( IsNotAlive() )
+    if( rBHelper.bDisposed || rBHelper.bInDispose )
         throw lang::DisposedException();
 }
 
diff --git a/svtools/source/control/calendar.cxx b/svtools/source/control/calendar.cxx
index 464a748..d51b05f 100644
--- a/svtools/source/control/calendar.cxx
+++ b/svtools/source/control/calendar.cxx
@@ -2275,18 +2275,13 @@ bool CalendarField::ShowDropDown( bool bShow )
     return true;
 }
 
-VclPtr<Calendar> CalendarField::CreateCalendar( vcl::Window* pParent )
-{
-    return VclPtr<Calendar>::Create( pParent, mnCalendarStyle | WB_TABSTOP );
-}
-
 Calendar* CalendarField::GetCalendar()
 {
     if ( !mpFloatWin )
     {
         mpFloatWin = VclPtr<ImplCFieldFloatWin>::Create( this );
         mpFloatWin->SetPopupModeEndHdl( LINK( this, CalendarField, ImplPopupModeEndHdl ) );
-        mpCalendar = CreateCalendar( mpFloatWin );
+        mpCalendar = VclPtr<Calendar>::Create( mpFloatWin, mnCalendarStyle | WB_TABSTOP );
         mpCalendar->SetPosPixel( Point() );
         mpCalendar->SetSelectHdl( LINK( this, CalendarField, ImplSelectHdl ) );
     }
diff --git a/svtools/source/control/filectrl.cxx b/svtools/source/control/filectrl.cxx
index 9164832..e0ccea2 100644
--- a/svtools/source/control/filectrl.cxx
+++ b/svtools/source/control/filectrl.cxx
@@ -172,11 +172,6 @@ void FileControl::Resize()
 }
 
 
-IMPL_LINK_NOARG(FileControl, ButtonHdl, Button*, void)
-{
-    ImplBrowseFile( );
-}
-
 void FileControl::GetFocus()
 {
     maEdit->GrabFocus();
@@ -199,7 +194,7 @@ void FileControl::Draw( OutputDevice* pDev, const Point& rPos, const Size& rSize
         GetEdit().SetStyle( nOldEditStyle );
 }
 
-void FileControl::ImplBrowseFile( )
+IMPL_LINK_NOARG(FileControl, ButtonHdl, Button*, void)
 {
     try
     {
diff --git a/svtools/source/control/toolbarmenu.cxx b/svtools/source/control/toolbarmenu.cxx
index 00eec20..ad4eaef 100644
--- a/svtools/source/control/toolbarmenu.cxx
+++ b/svtools/source/control/toolbarmenu.cxx
@@ -219,12 +219,6 @@ void ToolbarMenu_Impl::fireAccessibleEvent( short nEventId, const css::uno::Any&
 }
 
 
-bool ToolbarMenu_Impl::hasAccessibleListeners()
-{
-    return( mxAccessible.is() && mxAccessible->HasAccessibleListeners() );
-}
-
-
 sal_Int32 ToolbarMenu_Impl::getAccessibleChildCount() throw (RuntimeException)
 {
     sal_Int32 nCount = 0;
@@ -355,7 +349,7 @@ void ToolbarMenu_Impl::clearAccessibleSelection()
 
 void ToolbarMenu_Impl::notifyHighlightedEntry()
 {
-    if( hasAccessibleListeners() )
+    if( mxAccessible.is() && mxAccessible->HasAccessibleListeners() )
     {
         ToolbarMenuEntry* pEntry = implGetEntry( mnHighlightedEntry );
         if( pEntry && pEntry->mbEnabled && (pEntry->mnEntryId != TITLE_ID) )
diff --git a/svtools/source/control/toolbarmenuacc.cxx b/svtools/source/control/toolbarmenuacc.cxx
index 30484f4..2fda0af 100644
--- a/svtools/source/control/toolbarmenuacc.cxx
+++ b/svtools/source/control/toolbarmenuacc.cxx
@@ -62,20 +62,13 @@ IMPL_LINK( ToolbarMenuAcc, WindowEventListener, VclWindowEvent&, rEvent, void )
      * might have been destroyed by the previous VCLEventListener (if no AT tool
      * is running), e.g. sub-toolbars in impress.
      */
-    if ( mpParent && (rEvent.GetId() != VCLEVENT_WINDOW_ENDPOPUPMODE) )
-    {
-        DBG_ASSERT( rEvent.GetWindow(), "Window???" );
-        if( !rEvent.GetWindow()->IsAccessibilityEventsSuppressed() || ( rEvent.GetId() == VCLEVENT_OBJECT_DYING ) )
-        {
-            ProcessWindowEvent( rEvent );
-        }
-    }
-}
-
+    if ( !mpParent || (rEvent.GetId() == VCLEVENT_WINDOW_ENDPOPUPMODE) )
+        return;
+    DBG_ASSERT( rEvent.GetWindow(), "Window???" );
+    if( rEvent.GetWindow()->IsAccessibilityEventsSuppressed() && ( rEvent.GetId() != VCLEVENT_OBJECT_DYING ) )
+        return;
 
-void ToolbarMenuAcc::ProcessWindowEvent( const VclWindowEvent& rVclWindowEvent )
-{
-    switch ( rVclWindowEvent.GetId() )
+    switch ( rEvent.GetId() )
     {
         case VCLEVENT_OBJECT_DYING:
         {
diff --git a/svtools/source/control/toolbarmenuimp.hxx b/svtools/source/control/toolbarmenuimp.hxx
index 6943fba..af4ac49 100644
--- a/svtools/source/control/toolbarmenuimp.hxx
+++ b/svtools/source/control/toolbarmenuimp.hxx
@@ -165,8 +165,6 @@ private:
     /// The current FOCUSED state.
     bool mbIsFocused;
 
-    void ProcessWindowEvent( const VclWindowEvent& rVclWindowEvent );
-
     /** Tell all listeners that the object is dying.  This callback is
         usually called from the WeakComponentImplHelper class.
     */
@@ -262,7 +260,6 @@ struct ToolbarMenu_Impl
     void setAccessible( ToolbarMenuAcc* pAccessible );
 
     void fireAccessibleEvent( short nEventId, const css::uno::Any& rOldValue, const css::uno::Any& rNewValue );
-    bool hasAccessibleListeners();
 
     sal_Int32 getAccessibleChildCount() throw (css::uno::RuntimeException);
     css::uno::Reference< css::accessibility::XAccessible > getAccessibleChild( sal_Int32 index ) throw (css::lang::IndexOutOfBoundsException, css::uno::RuntimeException);
diff --git a/svtools/source/dialogs/PlaceEditDialog.cxx b/svtools/source/dialogs/PlaceEditDialog.cxx
index 893ccb7..33e1f20 100644
--- a/svtools/source/dialogs/PlaceEditDialog.cxx
+++ b/svtools/source/dialogs/PlaceEditDialog.cxx
@@ -221,32 +221,6 @@ void PlaceEditDialog::InitDetails( )
     SelectTypeHdl( *m_pLBServerType );
 }
 
-void PlaceEditDialog::UpdateLabel( )
-{
-    if( !bLabelChanged )
-    {
-        if( !m_pEDUsername->GetText().isEmpty( ) )
-        {
-            OUString sLabel = SvtResId( STR_SVT_DEFAULT_SERVICE_LABEL );
-            OUString sUser = m_pEDUsername->GetText();
-
-            int nLength = sUser.indexOf( '@' );
-            if( nLength < 0 )
-                nLength = sUser.getLength();
-
-            sLabel = sLabel.replaceFirst( "$user$", sUser.copy( 0, nLength ) );
-            sLabel = sLabel.replaceFirst( "$service$", m_pLBServerType->GetSelectEntry() );
-
-            m_pEDServerName->SetText( sLabel );
-            bLabelChanged = false;
-        }
-        else
-        {
-            m_pEDServerName->SetText( m_pLBServerType->GetSelectEntry( ) );
-        }
-    }
-}
-
 IMPL_LINK( PlaceEditDialog, OKHdl, Button*, /*pBtn*/, void)
 {
     if ( m_xCurrentDetails.get() )
@@ -290,7 +264,28 @@ IMPL_LINK( PlaceEditDialog, DelHdl, Button*, /*pButton*/, void)
 
 IMPL_LINK_NOARG( PlaceEditDialog, EditHdl, DetailsContainer*, void )
 {
-    UpdateLabel( );
+    if( !bLabelChanged )
+    {
+        if( !m_pEDUsername->GetText().isEmpty( ) )
+        {
+            OUString sLabel = SvtResId( STR_SVT_DEFAULT_SERVICE_LABEL );
+            OUString sUser = m_pEDUsername->GetText();
+
+            int nLength = sUser.indexOf( '@' );
+            if( nLength < 0 )
+                nLength = sUser.getLength();
+
+            sLabel = sLabel.replaceFirst( "$user$", sUser.copy( 0, nLength ) );
+            sLabel = sLabel.replaceFirst( "$service$", m_pLBServerType->GetSelectEntry() );
+
+            m_pEDServerName->SetText( sLabel );
+            bLabelChanged = false;
+        }
+        else
+        {
+            m_pEDServerName->SetText( m_pLBServerType->GetSelectEntry( ) );
+        }
+    }
 
     OUString sUrl = GetServerUrl( );
     OUString sName = m_pEDServerName->GetText().trim( );
diff --git a/svtools/source/dialogs/addresstemplate.cxx b/svtools/source/dialogs/addresstemplate.cxx
index 3fea725..2dd42da 100644
--- a/svtools/source/dialogs/addresstemplate.cxx
+++ b/svtools/source/dialogs/addresstemplate.cxx
@@ -248,7 +248,6 @@ namespace svt
 
     protected:
         css::uno::Any   getProperty(const OUString& _rLocalName) const;
-        css::uno::Any   getProperty(const sal_Char* _pLocalName) const;
 
         OUString        getStringProperty(const sal_Char* _pLocalName) const;
         OUString        getStringProperty(const OUString& _rLocalName) const;
@@ -322,12 +321,6 @@ void AssignmentPersistentData::ImplCommit()
     }
 
 
-    Any AssignmentPersistentData::getProperty(const sal_Char* _pLocalName) const
-    {
-        return getProperty(OUString::createFromAscii(_pLocalName));
-    }
-
-
     Any AssignmentPersistentData::getProperty(const OUString& _rLocalName) const
     {
         Sequence< OUString > aProperties(&_rLocalName, 1);
@@ -348,7 +341,7 @@ void AssignmentPersistentData::ImplCommit()
     OUString AssignmentPersistentData::getStringProperty(const sal_Char* _pLocalName) const
     {
         OUString sReturn;
-        getProperty( _pLocalName ) >>= sReturn;
+        getProperty(OUString::createFromAscii(_pLocalName)) >>= sReturn;
         return sReturn;
     }
 
diff --git a/svtools/source/dialogs/wizdlg.cxx b/svtools/source/dialogs/wizdlg.cxx
index e7e7508..2829ce2 100644
--- a/svtools/source/dialogs/wizdlg.cxx
+++ b/svtools/source/dialogs/wizdlg.cxx
@@ -105,14 +105,9 @@ void WizardDialog::ImplCalcSize( Size& rSize )
     }
 }
 
-bool WizardDialog::hasWizardPendingLayout() const
-{
-    return maWizardLayoutIdle.IsActive();
-}
-
 void WizardDialog::queue_resize(StateChangedType /*eReason*/)
 {
-    if (hasWizardPendingLayout())
+    if (maWizardLayoutIdle.IsActive())
         return;
     if (IsInClose())
         return;
diff --git a/svtools/source/graphic/descriptor.cxx b/svtools/source/graphic/descriptor.cxx
index e25d870..ca4b434 100644
--- a/svtools/source/graphic/descriptor.cxx
+++ b/svtools/source/graphic/descriptor.cxx
@@ -153,21 +153,6 @@ void GraphicDescriptor::implCreate( SvStream& rIStm, const OUString* pURL )
 }
 
 
-OUString GraphicDescriptor::getImplementationName_Static()
-    throw()
-{
-    return OUString( "com.sun.star.comp.graphic.GraphicDescriptor"  );
-}
-
-
-uno::Sequence< OUString > GraphicDescriptor::getSupportedServiceNames_Static()
-    throw(  )
-{
-    uno::Sequence< OUString > aSeq { "com.sun.star.graphic.GraphicDescriptor" };
-    return aSeq;
-}
-
-
 uno::Any SAL_CALL GraphicDescriptor::queryAggregation( const uno::Type & rType )
     throw( uno::RuntimeException, std::exception )
 {
@@ -214,7 +199,7 @@ void SAL_CALL GraphicDescriptor::release()
 OUString SAL_CALL GraphicDescriptor::getImplementationName()
     throw( uno::RuntimeException, std::exception )
 {
-    return getImplementationName_Static();
+    return OUString( "com.sun.star.comp.graphic.GraphicDescriptor"  );
 }
 
 sal_Bool SAL_CALL GraphicDescriptor::supportsService( const OUString& ServiceName )
@@ -227,7 +212,7 @@ sal_Bool SAL_CALL GraphicDescriptor::supportsService( const OUString& ServiceNam
 uno::Sequence< OUString > SAL_CALL GraphicDescriptor::getSupportedServiceNames()
     throw( uno::RuntimeException, std::exception )
 {
-    return getSupportedServiceNames_Static();
+    return { "com.sun.star.graphic.GraphicDescriptor" };
 }
 
 
diff --git a/svtools/source/graphic/descriptor.hxx b/svtools/source/graphic/descriptor.hxx
index 4fbf441..029a1d8 100644
--- a/svtools/source/graphic/descriptor.hxx
+++ b/svtools/source/graphic/descriptor.hxx
@@ -74,9 +74,6 @@ public:
     void init( const OUString& rURL );
     void init( const css::uno::Reference< css::io::XInputStream >& rxIStm, const OUString& rURL );
 
-    static OUString getImplementationName_Static() throw();
-    static css::uno::Sequence< OUString >  getSupportedServiceNames_Static() throw();
-
 protected:
 
     static ::comphelper::PropertySetInfo* createPropertySetInfo();
diff --git a/svtools/source/graphic/graphic.cxx b/svtools/source/graphic/graphic.cxx
index 2d18804..91d7847 100644
--- a/svtools/source/graphic/graphic.cxx
+++ b/svtools/source/graphic/graphic.cxx
@@ -88,23 +88,10 @@ void SAL_CALL Graphic::release() throw()
     unographic::GraphicDescriptor::release();
 }
 
-OUString Graphic::getImplementationName_Static() throw()
-{
-    return OUString( "com.sun.star.comp.graphic.Graphic" );
-}
-
-
-uno::Sequence< OUString > Graphic::getSupportedServiceNames_Static()
-    throw()
-{
-    uno::Sequence< OUString > aSeq { "com.sun.star.graphic.Graphic" };
-    return aSeq;
-}
-
 OUString SAL_CALL Graphic::getImplementationName()
     throw( uno::RuntimeException, std::exception )
 {
-    return getImplementationName_Static();
+    return OUString( "com.sun.star.comp.graphic.Graphic" );
 }
 
 sal_Bool SAL_CALL Graphic::supportsService( const OUString& rServiceName )
@@ -117,7 +104,7 @@ uno::Sequence< OUString > SAL_CALL Graphic::getSupportedServiceNames()
     throw( uno::RuntimeException, std::exception )
 {
     uno::Sequence< OUString >    aRet( ::unographic::GraphicDescriptor::getSupportedServiceNames() );
-    uno::Sequence< OUString >    aNew( getSupportedServiceNames_Static() );
+    uno::Sequence< OUString >    aNew { "com.sun.star.graphic.Graphic" };
     sal_Int32                           nOldCount = aRet.getLength();
 
     aRet.realloc( nOldCount + aNew.getLength() );
diff --git a/svtools/source/graphic/graphic.hxx b/svtools/source/graphic/graphic.hxx
index ee34096..e097c5a 100644
--- a/svtools/source/graphic/graphic.hxx
+++ b/svtools/source/graphic/graphic.hxx
@@ -47,8 +47,6 @@ public:
     void init( const ::Graphic& rGraphic ) throw();
 
     static const ::Graphic* getImplementation( const css::uno::Reference< css::uno::XInterface >& rxIFace ) throw();
-    static OUString getImplementationName_Static() throw();
-    static css::uno::Sequence< OUString >  getSupportedServiceNames_Static() throw();
 
 protected:
 
diff --git a/svtools/source/graphic/grfmgr.cxx b/svtools/source/graphic/grfmgr.cxx
index 214f378..a588edf 100644
--- a/svtools/source/graphic/grfmgr.cxx
+++ b/svtools/source/graphic/grfmgr.cxx
@@ -650,7 +650,7 @@ bool GraphicObject::StartAnimation( OutputDevice* pOut, const Point& rPt, const
                     delete mpSimpleCache;
 
                 mpSimpleCache = new GrfSimpleCacheObj( GetTransformedGraphic( &aAttr ), aAttr );
-                mpSimpleCache->maGraphic.SetAnimationNotifyHdl( GetAnimationNotifyHdl() );
+                mpSimpleCache->maGraphic.SetAnimationNotifyHdl( GetGraphic().GetAnimationNotifyHdl() );
             }
 
             mpSimpleCache->maGraphic.StartAnimation( pOut, aPt, aSz, nExtraData, pFirstFrameOutDev );
@@ -681,7 +681,12 @@ const Graphic& GraphicObject::GetGraphic() const
     //fdo#50697 If we've been asked to provide the graphic, then reset
     //the cache timeout to start from now and not remain at the
     //time of creation
-    pThis->restartSwapOutTimer();
+    // restart SwapOut timer; this is like touching in a cache to reset to the full timeout value
+    if( pThis->mpSwapOutTimer && pThis->mpSwapOutTimer->IsActive() )
+    {
+        pThis->mpSwapOutTimer->Stop();
+        pThis->mpSwapOutTimer->Start();
+    }
 
     return maGraphic;
 }
@@ -1183,14 +1188,5 @@ basegfx::B2DVector GraphicObject::calculateCropScaling(
     return basegfx::B2DVector(fFactorX,fFactorY);
 }
 
-// restart SwapOut timer
-void GraphicObject::restartSwapOutTimer() const
-{
-    if( mpSwapOutTimer && mpSwapOutTimer->IsActive() )
-    {
-        mpSwapOutTimer->Stop();
-        mpSwapOutTimer->Start();
-    }
-}
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/svtools/source/inc/svimpbox.hxx b/svtools/source/inc/svimpbox.hxx
index b29b49d..4a892cf 100644
--- a/svtools/source/inc/svimpbox.hxx
+++ b/svtools/source/inc/svimpbox.hxx
@@ -189,9 +189,6 @@ private:
     // if element at cursor can be expanded in general
     bool IsExpandable() const;
 
-    // if element at cursor can be expanded at this moment
-    bool IsNowExpandable() const;
-
     static  void        implInitDefaultNodeImages();
 
     void UpdateStringSorter();
diff --git a/svtools/source/inc/unoiface.hxx b/svtools/source/inc/unoiface.hxx
index adb0485..fbf987a 100644
--- a/svtools/source/inc/unoiface.hxx
+++ b/svtools/source/inc/unoiface.hxx
@@ -314,8 +314,7 @@ public:
     void SAL_CALL setStrictFormat( sal_Bool bStrict ) throw(css::uno::RuntimeException, std::exception) override;
     sal_Bool SAL_CALL isStrictFormat(  ) throw(css::uno::RuntimeException, std::exception) override;
 
-    static void     ImplGetPropertyIds( std::vector< sal_uInt16 > &aIds );
-    virtual void    GetPropertyIds( std::vector< sal_uInt16 > &aIds ) override { return ImplGetPropertyIds( aIds ); }
+    virtual void    GetPropertyIds( std::vector< sal_uInt16 > &aIds ) override;
 };
 
 
diff --git a/svtools/source/misc/dialogcontrolling.cxx b/svtools/source/misc/dialogcontrolling.cxx
index ea5e818..11e6d50 100644
--- a/svtools/source/misc/dialogcontrolling.cxx
+++ b/svtools/source/misc/dialogcontrolling.cxx
@@ -104,18 +104,13 @@ namespace svt
 
     IMPL_LINK( DialogController, OnWindowEvent, VclWindowEvent&, _rEvent, void )
     {
-        if ( m_pImpl->pEventFilter->payAttentionTo( _rEvent ) )
-            impl_updateAll( _rEvent );
-    }
-
-
-    void DialogController::impl_updateAll( const VclWindowEvent& _rTriggerEvent )
-    {
+        if ( !m_pImpl->pEventFilter->payAttentionTo( _rEvent ) )
+            return;
         for ( auto loop = m_pImpl->aConcernedWindows.begin();
                 loop != m_pImpl->aConcernedWindows.end();
                 ++loop
             )
-            impl_update( _rTriggerEvent, *(*loop) );
+            impl_update( _rEvent, *(*loop) );
     }
 
 
diff --git a/svtools/source/misc/templatefoldercache.cxx b/svtools/source/misc/templatefoldercache.cxx
index d2e1be5..6979be1 100644
--- a/svtools/source/misc/templatefoldercache.cxx
+++ b/svtools/source/misc/templatefoldercache.cxx
@@ -420,7 +420,6 @@ namespace svt
 
         bool        implReadFolder( const ::rtl::Reference< TemplateContent >& _rxRoot );
 
-        static  OUString getCacheFileName();
         static  sal_Int32   getMagicNumber();
         static  void        normalize( TemplateFolderContent& _rState );
 
@@ -463,12 +462,6 @@ namespace svt
     }
 
 
-    OUString TemplateFolderCacheImpl::getCacheFileName()
-    {
-        return OUString(".templdir.cache");
-    }
-
-
     void TemplateFolderCacheImpl::normalize( TemplateFolderContent& _rState )
     {
         SubContentSort()( _rState );
@@ -723,7 +716,7 @@ namespace svt
         }
 
         // append our name
-        aStorageURL.Append( getCacheFileName() );
+        aStorageURL.Append( ".templdir.cache" );
 
         // open the stream
         m_pCacheStream = UcbStreamHelper::CreateStream( aStorageURL.GetMainURL( INetURLObject::DECODE_TO_IURI ),
diff --git a/svtools/source/table/tabledatawindow.cxx b/svtools/source/table/tabledatawindow.cxx
index 8240f5f..d4fd7cd 100644
--- a/svtools/source/table/tabledatawindow.cxx
+++ b/svtools/source/table/tabledatawindow.cxx
@@ -56,11 +56,6 @@ namespace svt { namespace table
         m_rTableControl.doPaintContent(rRenderContext, rUpdateRect);
     }
 
-    void TableDataWindow::SetBackground( const Wallpaper& rColor )
-    {
-        Window::SetBackground( rColor );
-    }
-
     void TableDataWindow::RequestHelp( const HelpEvent& rHEvt )
     {
         HelpEventMode const nHelpMode = rHEvt.GetMode();
diff --git a/svtools/source/table/tabledatawindow.hxx b/svtools/source/table/tabledatawindow.hxx
index ee1300e..a3e2273 100644
--- a/svtools/source/table/tabledatawindow.hxx
+++ b/svtools/source/table/tabledatawindow.hxx
@@ -57,8 +57,6 @@ namespace svt { namespace table
         virtual bool        Notify(NotifyEvent& rNEvt) override;
         virtual void        RequestHelp( const HelpEvent& rHEvt ) override;
 
-        void                SetBackground(const Wallpaper& rColor);
-
     private:
         static void impl_hideTipWindow();
     };
diff --git a/svtools/source/uno/unoevent.cxx b/svtools/source/uno/unoevent.cxx
index 7cc356a..ce3ab34 100644
--- a/svtools/source/uno/unoevent.cxx
+++ b/svtools/source/uno/unoevent.cxx
@@ -513,16 +513,6 @@ SvMacroTableEventDescriptor::SvMacroTableEventDescriptor(
     const SvEventDescription* pSupportedMacroItems) :
         SvDetachedEventDescriptor(pSupportedMacroItems)
 {
-    copyMacrosFromTable(rMacroTable);
-}
-
-SvMacroTableEventDescriptor::~SvMacroTableEventDescriptor()
-{
-}
-
-void SvMacroTableEventDescriptor::copyMacrosFromTable(
-    const SvxMacroTableDtor& rMacroTable)
-{
     for(sal_Int16 i = 0; mpSupportedMacroItems[i].mnEvent != 0; i++)
     {
         const sal_uInt16 nEvent = mpSupportedMacroItems[i].mnEvent;
@@ -530,7 +520,10 @@ void SvMacroTableEventDescriptor::copyMacrosFromTable(
         if (nullptr != pMacro)
             replaceByName(nEvent, *pMacro);
     }
+}
 
+SvMacroTableEventDescriptor::~SvMacroTableEventDescriptor()
+{
 }
 
 void SvMacroTableEventDescriptor::copyMacrosIntoTable(
diff --git a/svtools/source/uno/unoiface.cxx b/svtools/source/uno/unoiface.cxx
index f1df7df..f0a914a 100644
--- a/svtools/source/uno/unoiface.cxx
+++ b/svtools/source/uno/unoiface.cxx
@@ -1796,7 +1796,7 @@ sal_Bool SVTXNumericField::isStrictFormat() throw(css::uno::RuntimeException, st
     return pField && pField->IsStrictFormat();
 }
 
-void SVTXNumericField::ImplGetPropertyIds( std::vector< sal_uInt16 > &rIds )
+void SVTXNumericField::GetPropertyIds( std::vector< sal_uInt16 > &rIds )
 {
     SVTXFormattedField::ImplGetPropertyIds( rIds );
 }


More information about the Libreoffice-commits mailing list