loplugin:unusedfields in sfx2

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

6 files changed, 10 insertions, 39 deletions

diff --git a/sfx2/source/appl/lnkbase2.cxx b/sfx2/source/appl/lnkbase2.cxx
index d3aa40b38090..c4bd1ac7382c 100644
--- a/sfx2/source/appl/lnkbase2.cxx
+++ b/sfx2/source/appl/lnkbase2.cxx
@@ -126,7 +126,7 @@ SvBaseLink::SvBaseLink()
 {
     nObjType = OBJECT_CLIENT_SO;
     pImplData = new ImplBaseLinkData;
-    bVisible = bSynchron = bUseCache = true;
+    bVisible = bSynchron = true;
     bWasLastEditOK = false;
 }
 
@@ -137,7 +137,7 @@ SvBaseLink::SvBaseLink( SfxLinkUpdateMode nUpdateMode, SotClipboardFormatId nCon
 {
     nObjType = OBJECT_CLIENT_SO;
     pImplData = new ImplBaseLinkData;
-    bVisible = bSynchron = bUseCache = true;
+    bVisible = bSynchron = true;
     bWasLastEditOK = false;
 
     // It is going to be a Ole-Link,
@@ -184,7 +184,7 @@ SvBaseLink::SvBaseLink( const OUString& rLinkName, sal_uInt16 nObjectType, SvLin
     : pImpl()
     , m_bIsReadOnly(false)
 {
-    bVisible = bSynchron = bUseCache = true;
+    bVisible = bSynchron = true;
     bWasLastEditOK = false;
     aLinkName = rLinkName;
     pImplData = new ImplBaseLinkData;
diff --git a/sfx2/source/appl/workwin.cxx b/sfx2/source/appl/workwin.cxx
index 8e662c6f44b2..9bab15397683 100644
--- a/sfx2/source/appl/workwin.cxx
+++ b/sfx2/source/appl/workwin.cxx
@@ -1000,7 +1000,6 @@ void SfxWorkWindow::SetObjectBar_Impl(sal_uInt16 nPos, SfxVisibilityFlags nFlags
     }
 
     SfxObjectBar_Impl aObjBar;
-    aObjBar.pIFace = pIFace;
     aObjBar.eId = eId;
     aObjBar.nPos = nPos;
     aObjBar.nMode = nFlags;
diff --git a/sfx2/source/bastyp/frmhtml.cxx b/sfx2/source/bastyp/frmhtml.cxx
index 094a616b45e2..f49cbd8e0b5f 100644
--- a/sfx2/source/bastyp/frmhtml.cxx
+++ b/sfx2/source/bastyp/frmhtml.cxx
@@ -60,13 +60,6 @@ void SfxFrameHTMLParser::ParseFrameOptions(
     {
         switch( rOption.GetToken() )
         {
-        case HtmlOptionId::BORDERCOLOR:
-            {
-                Color aColor;
-                rOption.GetColor( aColor );
-                pFrame->SetWallpaper( Wallpaper( aColor ) );
-                break;
-            }
         case HtmlOptionId::SRC:
             pFrame->SetURL(
                     INetURLObject::GetAbsURL(
@@ -102,9 +95,6 @@ void SfxFrameHTMLParser::ParseFrameOptions(
             pFrame->SetFrameBorder( bBorder );
             break;
         }
-        case HtmlOptionId::NORESIZE:
-            pFrame->SetResizable( false );
-            break;
         default:
             break;
         }
diff --git a/sfx2/source/doc/frmdescr.cxx b/sfx2/source/doc/frmdescr.cxx
index bf5685bf5b08..143c8742b2f4 100644
--- a/sfx2/source/doc/frmdescr.cxx
+++ b/sfx2/source/doc/frmdescr.cxx
@@ -23,18 +23,13 @@
 
 #include <sfx2/frmdescr.hxx>
 #include <sfx2/app.hxx>
+#include <memory>
 
 struct SfxFrameDescriptor_Impl
 {
-    Wallpaper*  pWallpaper;
-    SfxItemSet* pArgs;
+    std::unique_ptr<SfxItemSet> pArgs;
 
-    SfxFrameDescriptor_Impl() : pWallpaper( nullptr ), pArgs( nullptr ) {}
-    ~SfxFrameDescriptor_Impl()
-    {
-        delete pWallpaper;
-        delete pArgs;
-    }
+    SfxFrameDescriptor_Impl() {}
 };
 
 SfxFrameDescriptor::SfxFrameDescriptor() :
@@ -42,7 +37,6 @@ SfxFrameDescriptor::SfxFrameDescriptor() :
     eScroll( ScrollingMode::Auto ),
     bHasBorder( true ),
     bHasBorderSet( false ),
-    bResizeVertical( true ),
     pImpl( new SfxFrameDescriptor_Impl )
 {
 }
@@ -54,8 +48,8 @@ SfxFrameDescriptor::~SfxFrameDescriptor()
 SfxItemSet* SfxFrameDescriptor::GetArgs()
 {
     if( !pImpl->pArgs )
-        pImpl->pArgs = new SfxAllItemSet( SfxGetpApp()->GetPool() );
-    return pImpl->pArgs;
+        pImpl->pArgs.reset( new SfxAllItemSet( SfxGetpApp()->GetPool() ) );
+    return pImpl->pArgs.get();
 }
 
 void SfxFrameDescriptor::SetURL( const OUString& rURL )
@@ -71,12 +65,4 @@ void SfxFrameDescriptor::SetActualURL( const OUString& rURL )
         pImpl->pArgs->ClearItem();
 }
 
-void SfxFrameDescriptor::SetWallpaper( const Wallpaper& rWallpaper )
-{
-    DELETEZ( pImpl->pWallpaper );
-
-    if ( rWallpaper.GetStyle() != WallpaperStyle::NONE )
-        pImpl->pWallpaper = new Wallpaper( rWallpaper );
-}
-
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sfx2/source/inc/workwin.hxx b/sfx2/source/inc/workwin.hxx
index 7eb7e8a649e7..6f1eb124f1b2 100644
--- a/sfx2/source/inc/workwin.hxx
+++ b/sfx2/source/inc/workwin.hxx
@@ -49,14 +49,12 @@ struct SfxObjectBar_Impl
     SfxVisibilityFlags nMode; // special visibility flags
     sal_uInt16         nPos;
     bool               bDestroy;
-    SfxInterface*      pIFace;
 
     SfxObjectBar_Impl() :
         eId(ToolbarId::None),
         nMode(SfxVisibilityFlags::Invisible),
         nPos(0),
-        bDestroy(false),
-        pIFace(nullptr)
+        bDestroy(false)
     {}
 };
 
diff --git a/sfx2/source/sidebar/DeckLayouter.cxx b/sfx2/source/sidebar/DeckLayouter.cxx
index 5a37474148c5..b0d177ec7406 100644
--- a/sfx2/source/sidebar/DeckLayouter.cxx
+++ b/sfx2/source/sidebar/DeckLayouter.cxx
@@ -47,11 +47,10 @@ namespace {
         css::ui::LayoutSize maLayoutSize;
         sal_Int32 mnDistributedHeight;
         sal_Int32 mnWeight;
-        sal_Int32 mnPanelIndex;
         bool mbShowTitleBar;
 
         LayoutItem()
-            : mpPanel(),maLayoutSize(0,0,0),mnDistributedHeight(0),mnWeight(0),mnPanelIndex(0),mbShowTitleBar(true)
+            : mpPanel(),maLayoutSize(0,0,0),mnDistributedHeight(0),mnWeight(0),mbShowTitleBar(true)
         {}
     };
     tools::Rectangle LayoutPanels (
@@ -115,7 +114,6 @@ void DeckLayouter::LayoutDeck (
         for (sal_Int32 nIndex(0),nCount(rPanels.size()); nIndex<nCount; ++nIndex)
         {
             aLayoutItems[nIndex].mpPanel = rPanels[nIndex];
-            aLayoutItems[nIndex].mnPanelIndex = nIndex;
         }
         aBox = LayoutPanels(
             aBox,