loplugin:unusedfields in framework

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

12 files changed, 0 insertions, 83 deletions

diff --git a/framework/source/accelerators/presethandler.cxx b/framework/source/accelerators/presethandler.cxx
index 34c94719e6df..fc5f0a19ca30 100644
--- a/framework/source/accelerators/presethandler.cxx
+++ b/framework/source/accelerators/presethandler.cxx
@@ -77,25 +77,18 @@ PresetHandler::PresetHandler(const css::uno::Reference< css::uno::XComponentCont
     : m_xContext(xContext)
     , m_eConfigType(E_GLOBAL)
     , m_lDocumentStorages()
-    , m_aLanguageTag(LANGUAGE_USER_PRIV_NOTRANSLATE)
 {
 }
 
 PresetHandler::PresetHandler(const PresetHandler& rCopy)
-    : m_aLanguageTag( rCopy.m_aLanguageTag)
 {
     m_xContext              = rCopy.m_xContext;
     m_eConfigType           = rCopy.m_eConfigType;
-    m_sResourceType         = rCopy.m_sResourceType;
-    m_sModule               = rCopy.m_sModule;
     m_xWorkingStorageShare  = rCopy.m_xWorkingStorageShare;
     m_xWorkingStorageNoLang = rCopy.m_xWorkingStorageNoLang;
     m_xWorkingStorageUser   = rCopy.m_xWorkingStorageUser;
-    m_lPresets              = rCopy.m_lPresets;
-    m_lTargets              = rCopy.m_lTargets;
     m_lDocumentStorages     = rCopy.m_lDocumentStorages;
     m_sRelPathShare         = rCopy.m_sRelPathShare;
-    m_sRelPathNoLang        = rCopy.m_sRelPathNoLang;
     m_sRelPathUser          = rCopy.m_sRelPathUser;
 }
 
@@ -324,9 +317,6 @@ void PresetHandler::connectToResource(      PresetHandler::EConfigType
     {
         SolarMutexGuard g;
         m_eConfigType   = eConfigType;
-        m_sResourceType = sResource;
-        m_sModule       = sModule;
-        m_aLanguageTag  = rLanguageTag;
     }
 
     css::uno::Reference< css::embed::XStorage > xShare;
@@ -496,10 +486,7 @@ void PresetHandler::connectToResource(      PresetHandler::EConfigType
         m_xWorkingStorageShare = xShare;
         m_xWorkingStorageNoLang= xNoLang;
         m_xWorkingStorageUser  = xUser;
-        m_lPresets             = lPresets;
-        m_lTargets             = lTargets;
         m_sRelPathShare        = sRelPathShare;
-        m_sRelPathNoLang       = sRelPathNoLang;
         m_sRelPathUser         = sRelPathUser;
     }
 
diff --git a/framework/source/helper/titlebarupdate.cxx b/framework/source/helper/titlebarupdate.cxx
index c75e8a32d974..0ab7dee17235 100644
--- a/framework/source/helper/titlebarupdate.cxx
+++ b/framework/source/helper/titlebarupdate.cxx
@@ -186,7 +186,6 @@ bool TitleBarUpdate::implst_getModuleInfo(const css::uno::Reference< css::frame:
         rInfo.sID = xModuleManager->identify(xFrame);
         ::comphelper::SequenceAsHashMap lProps    = xModuleManager->getByName (rInfo.sID);
 
-        rInfo.sUIName = lProps.getUnpackedValueOrDefault (OFFICEFACTORY_PROPNAME_ASCII_UINAME, OUString());
         rInfo.nIcon   = lProps.getUnpackedValueOrDefault (OFFICEFACTORY_PROPNAME_ASCII_ICON  , INVALID_ICON_ID  );
 
         // Note: If we could retrieve a module id ... everything is OK.
diff --git a/framework/source/helper/vclstatusindicator.cxx b/framework/source/helper/vclstatusindicator.cxx
index c2eb170bcc8f..8173c3b48e8b 100644
--- a/framework/source/helper/vclstatusindicator.cxx
+++ b/framework/source/helper/vclstatusindicator.cxx
@@ -60,7 +60,6 @@ void SAL_CALL VCLStatusIndicator::start(const OUString& sText ,
     pParentWindow->Invalidate(InvalidateFlags::Children);
     pParentWindow->Flush();
 
-    m_sText  = sText;
     m_nRange = nRange;
     m_nValue = 0;
 }
@@ -79,7 +78,6 @@ void SAL_CALL VCLStatusIndicator::end()
 {
     SolarMutexGuard aSolarGuard;
 
-    m_sText.clear();
     m_nRange = 0;
     m_nValue = 0;
 
@@ -95,7 +93,6 @@ void SAL_CALL VCLStatusIndicator::end()
 void SAL_CALL VCLStatusIndicator::setText(const OUString& sText)
 {
     SolarMutexGuard aSolarGuard;
-    m_sText = sText;
     if (m_pStatusBar)
         m_pStatusBar->SetText(sText);
 }
diff --git a/framework/source/inc/accelerators/presethandler.hxx b/framework/source/inc/accelerators/presethandler.hxx
index 56a2fc15d403..b41593d41bee 100644
--- a/framework/source/inc/accelerators/presethandler.hxx
+++ b/framework/source/inc/accelerators/presethandler.hxx
@@ -76,22 +76,6 @@ class PresetHandler
          */
         EConfigType m_eConfigType;
 
-        /** @short  specify the type of resource, which configuration sets
-                    must be provided here.
-
-            @descr  e.g. menubars, toolbars, accelerators
-         */
-        OUString m_sResourceType;
-
-        /** @short  specify the application module for a module
-                    dependent configuration.
-
-            @descr  Will be used only, if m_sResourceType is set to
-                    "module". Further it must be a valid module identifier
-                    then ...
-         */
-        OUString m_sModule;
-
         /** @short  if we run in document mode, we can't use the global root storages!
                     We have to use a special document storage explicitly. */
         StorageHolder m_lDocumentStorages;
@@ -124,24 +108,8 @@ class PresetHandler
          */
         css::uno::Reference< css::embed::XStorage > m_xWorkingStorageUser;
 
-        /** @short  knows the names of all presets inside the current
-                    working storage of the share layer. */
-        std::vector<OUString> m_lPresets;
-
-        /** @short  knows the names of all targets inside the current
-                    working storage of the user layer. */
-        std::vector<OUString> m_lTargets;
-
-        /** @short  it's the current office locale and will be used
-                    to handle localized presets.
-
-            @descr  Default is "x-no-translate" which disable any
-                    localized handling inside this class! */
-        LanguageTag m_aLanguageTag;
-
         /** @short  knows the relative path from the root. */
         OUString m_sRelPathShare;
-        OUString m_sRelPathNoLang;
         OUString m_sRelPathUser;
 
     // native interface
diff --git a/framework/source/jobs/job.cxx b/framework/source/jobs/job.cxx
index 574765ad3d22..14c6622a5d33 100644
--- a/framework/source/jobs/job.cxx
+++ b/framework/source/jobs/job.cxx
@@ -461,7 +461,6 @@ void Job::impl_reactForJobResult( /*IN*/ const css::uno::Any& aResult )
         (aAnalyzedResult.existPart(JobResult::E_DISPATCHRESULT))
        )
     {
-        m_aJobCfg.setResult(aAnalyzedResult);
         // Attention: Because the listener expect that the original object send this event ...
         // and we nor the job are the right ones ...
         // our user has set itself before. So we can fake this source address!
diff --git a/framework/source/jobs/jobdata.cxx b/framework/source/jobs/jobdata.cxx
index 823541875399..8984d618a1bc 100644
--- a/framework/source/jobs/jobdata.cxx
+++ b/framework/source/jobs/jobdata.cxx
@@ -83,7 +83,6 @@ JobData& JobData::operator=( const JobData& rCopy )
     m_sContext             = rCopy.m_sContext;
     m_sEvent               = rCopy.m_sEvent;
     m_lArguments           = rCopy.m_lArguments;
-    m_aLastExecutionResult = rCopy.m_aLastExecutionResult;
     return *this;
 }
 
@@ -264,30 +263,6 @@ void JobData::setJobConfig( const std::vector< css::beans::NamedValue >& lArgume
 }
 
 /**
-    @short      set a new execution result
-    @descr      Every executed job can have returned a result.
-                We set it here, so our user can use it may be later.
-                But the outside code can use it too, to analyze it and
-                adopt the configuration of this job too. Because the
-                result uses a protocol, which allow that. And we provide
-                right functionality to save it.
-
-    @param      aResult
-                    the result of last execution
- */
-void JobData::setResult( const JobResult& aResult )
-{
-    SolarMutexGuard g;
-
-    // overwrite the last saved result
-    m_aLastExecutionResult = aResult;
-
-    // Don't use his information to update
-    // e.g. the arguments of this job. It must be done
-    // from outside! Here we save this information only.
-}
-
-/**
     @short  set a new environment descriptor for this job
     @descr  It must(!) be done every time this container is initialized
             with new job datas e.g.: setAlias()/setEvent()/setService() ...
diff --git a/framework/source/layoutmanager/layoutmanager.cxx b/framework/source/layoutmanager/layoutmanager.cxx
index 0489804ed5fc..19d97528a263 100644
--- a/framework/source/layoutmanager/layoutmanager.cxx
+++ b/framework/source/layoutmanager/layoutmanager.cxx
@@ -435,7 +435,6 @@ void LayoutManager::implts_reset( bool bAttached )
 
         /* SAFE AREA ----------------------------------------------------------------------------------------------- */
         SolarMutexClearableGuard aWriteLock;
-        m_xModel = xModel;
         m_aDockingArea = awt::Rectangle();
         m_aModuleIdentifier = aModuleIdentifier;
         m_xModuleCfgMgr = xModCfgMgr;
diff --git a/framework/source/layoutmanager/toolbarlayoutmanager.cxx b/framework/source/layoutmanager/toolbarlayoutmanager.cxx
index 644eacb8d05b..9396171848f2 100644
--- a/framework/source/layoutmanager/toolbarlayoutmanager.cxx
+++ b/framework/source/layoutmanager/toolbarlayoutmanager.cxx
@@ -3268,7 +3268,6 @@ void SAL_CALL ToolbarLayoutManager::startDocking( const awt::DockingEvent& e )
     m_bDockingInProgress = bWinFound;
     m_aDockUIElement = aUIElement;
     m_aDockUIElement.m_bUserActive = true;
-    m_aStartDockMousePos = aMousePos;
 }
 
 awt::DockingData SAL_CALL ToolbarLayoutManager::docking( const awt::DockingEvent& e )
diff --git a/framework/source/layoutmanager/toolbarlayoutmanager.hxx b/framework/source/layoutmanager/toolbarlayoutmanager.hxx
index 12dfffe46e50..c24cee6ee53f 100644
--- a/framework/source/layoutmanager/toolbarlayoutmanager.hxx
+++ b/framework/source/layoutmanager/toolbarlayoutmanager.hxx
@@ -271,7 +271,6 @@ class ToolbarLayoutManager : public ::cppu::WeakImplHelper< css::awt::XDockableW
 
         UIElementVector                                                      m_aUIElements;
         UIElement                                                            m_aDockUIElement;
-        Point                                                                m_aStartDockMousePos;
         tools::Rectangle                                                            m_aDockingArea;
         tools::Rectangle                                                            m_aDockingAreaOffsets;
         DockingOperation                                                     m_eDockOperation;
diff --git a/framework/source/services/desktop.cxx b/framework/source/services/desktop.cxx
index 460f97a46e7b..7c3823e09f06 100644
--- a/framework/source/services/desktop.cxx
+++ b/framework/source/services/desktop.cxx
@@ -167,7 +167,6 @@ Desktop::Desktop( const css::uno::Reference< css::uno::XComponentContext >& xCon
         ,   m_xFramesHelper         (                                               )
         ,   m_xDispatchHelper       (                                               )
         ,   m_eLoadState            ( E_NOTSET                                      )
-        ,   m_aInteractionRequest   (                                               )
         ,   m_bSuspendQuickstartVeto( false                                     )
         ,   m_sName                 (                                               )
         ,   m_sTitle                (                                               )
@@ -1293,7 +1292,6 @@ void SAL_CALL Desktop::handle( const css::uno::Reference< css::task::XInteractio
     {
         SolarMutexGuard g;
         m_eLoadState          = E_INTERACTION;
-        m_aInteractionRequest = aRequest;
     }
 }
 
diff --git a/framework/source/uielement/addonstoolbarmanager.cxx b/framework/source/uielement/addonstoolbarmanager.cxx
index 7a6847e53844..eb3381ce6a04 100644
--- a/framework/source/uielement/addonstoolbarmanager.cxx
+++ b/framework/source/uielement/addonstoolbarmanager.cxx
@@ -262,7 +262,6 @@ void AddonsToolBarManager::FillToolbar( const Sequence< Sequence< PropertyValue
                 // Create TbRuntimeItemData to hold additional information we will need in the future
                 AddonsParams* pRuntimeItemData = new AddonsParams;
                 pRuntimeItemData->aImageId  = aImageId;
-                pRuntimeItemData->aTarget   = aTarget;
                 pRuntimeItemData->aControlType = aControlType;
                 pRuntimeItemData->nWidth    = nWidth;
                 m_pToolBar->SetItemData( nId, pRuntimeItemData );
diff --git a/framework/source/uielement/toolbarmerger.cxx b/framework/source/uielement/toolbarmerger.cxx
index 83416bc5daea..e66a31e77d85 100644
--- a/framework/source/uielement/toolbarmerger.cxx
+++ b/framework/source/uielement/toolbarmerger.cxx
@@ -229,7 +229,6 @@ ReferenceToolbarPathInfo ToolBarMerger::FindReferencePoint(
 {
     ReferenceToolbarPathInfo aResult;
     aResult.bResult  = false;
-    aResult.pToolbar = pToolbar;
     aResult.nPos     = ToolBox::ITEM_NOTFOUND;
 
     const ToolBox::ImplToolItems::size_type nSize( pToolbar->GetItemCount() );
@@ -627,7 +626,6 @@ void ToolBarMerger::CreateToolbarItem( ToolBox* pToolbar, ToolBox::ImplToolItems
     // Use the user data to store add-on specific data with the toolbar item
     AddonsParams* pAddonParams = new AddonsParams;
     pAddonParams->aImageId     = rItem.aImageIdentifier;
-    pAddonParams->aTarget      = rItem.aTarget;
     pAddonParams->aControlType = rItem.aControlType;
     pAddonParams->nWidth       = rItem.nWidth;
     pToolbar->SetItemData( nItemId, pAddonParams );