diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-08-03 11:39:22 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-08-03 13:13:17 +0200 |
commit | d5630adf33cc7d30a657e789e2bf0978028e0c1f (patch) | |
tree | 881a26b1980a617793f030028b0c8a69a8e672df /sd/source | |
parent | 70d0425dd4fffd51170ccf0ca12a28870b0b7b1a (diff) |
loplugin:checkunusedparams in various
Change-Id: I4f54940a9ebdcd47776b2a5f7bae6e49b633ee44
Reviewed-on: https://gerrit.libreoffice.org/40706
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sd/source')
-rw-r--r-- | sd/source/filter/eppt/pptexanimations.cxx | 26 | ||||
-rw-r--r-- | sd/source/filter/eppt/pptexanimations.hxx | 2 | ||||
-rw-r--r-- | sd/source/ui/framework/module/ResourceManager.cxx | 3 | ||||
-rw-r--r-- | sd/source/ui/framework/module/ResourceManager.hxx | 1 | ||||
-rw-r--r-- | sd/source/ui/sidebar/PanelFactory.cxx | 7 | ||||
-rw-r--r-- | sd/source/ui/sidebar/PanelFactory.hxx | 2 |
6 files changed, 18 insertions, 23 deletions
diff --git a/sd/source/filter/eppt/pptexanimations.cxx b/sd/source/filter/eppt/pptexanimations.cxx index 4c07402a1039..40e00edfff25 100644 --- a/sd/source/filter/eppt/pptexanimations.cxx +++ b/sd/source/filter/eppt/pptexanimations.cxx @@ -989,7 +989,7 @@ sal_Int16 AnimationExporter::exportAnimPropertySet( SvStream& rStrm, const Refer case css::presentation::EffectNodeType::TIMING_ROOT : nPPTNodeType = DFF_ANIM_NODE_TYPE_TIMING_ROOT; break; case css::presentation::EffectNodeType::INTERACTIVE_SEQUENCE: nPPTNodeType = DFF_ANIM_NODE_TYPE_INTERACTIVE_SEQ; break; } - exportAnimPropertyuInt32( rStrm, DFF_ANIM_NODE_TYPE, nPPTNodeType, TRANSLATE_NONE ); + exportAnimPropertyuInt32( rStrm, DFF_ANIM_NODE_TYPE, nPPTNodeType ); } } sal_uInt32 nPresetId = 0; @@ -1036,11 +1036,11 @@ sal_Int16 AnimationExporter::exportAnimPropertySet( SvStream& rStrm, const Refer } } if ( bPresetId ) - exportAnimPropertyuInt32( rStrm, DFF_ANIM_PRESET_ID, nPresetId, TRANSLATE_NONE ); + exportAnimPropertyuInt32( rStrm, DFF_ANIM_PRESET_ID, nPresetId ); if ( bPresetSubType ) - exportAnimPropertyuInt32( rStrm, DFF_ANIM_PRESET_SUB_TYPE, nPresetSubType, TRANSLATE_NONE ); + exportAnimPropertyuInt32( rStrm, DFF_ANIM_PRESET_SUB_TYPE, nPresetSubType ); if ( bPresetClass ) - exportAnimPropertyuInt32( rStrm, DFF_ANIM_PRESET_CLASS, nPresetClass, TRANSLATE_NONE ); + exportAnimPropertyuInt32( rStrm, DFF_ANIM_PRESET_CLASS, nPresetClass ); if ( pAny[ DFF_ANIM_ID ] ) { @@ -1058,7 +1058,7 @@ sal_Int16 AnimationExporter::exportAnimPropertySet( SvStream& rStrm, const Refer { sal_Int32 nRunTimeContext = 0; if ( *pAny[ DFF_ANIM_RUNTIMECONTEXT ] >>= nRunTimeContext ) - exportAnimPropertyuInt32( rStrm, DFF_ANIM_RUNTIMECONTEXT, nRunTimeContext, TRANSLATE_NONE ); + exportAnimPropertyuInt32( rStrm, DFF_ANIM_RUNTIMECONTEXT, nRunTimeContext ); } if ( pAny[ DFF_ANIM_PATH_EDIT_MODE ] ) { @@ -1072,7 +1072,7 @@ sal_Int16 AnimationExporter::exportAnimPropertySet( SvStream& rStrm, const Refer { bool bDirection = !xColor->getDirection(); - exportAnimPropertyuInt32( rStrm, DFF_ANIM_DIRECTION, bDirection ? 1 : 0, TRANSLATE_NONE ); + exportAnimPropertyuInt32( rStrm, DFF_ANIM_DIRECTION, bDirection ? 1 : 0 ); } } @@ -1080,14 +1080,14 @@ sal_Int16 AnimationExporter::exportAnimPropertySet( SvStream& rStrm, const Refer { sal_Int32 nOverride = 0; if ( *pAny[ DFF_ANIM_OVERRIDE ] >>= nOverride ) - exportAnimPropertyuInt32( rStrm, DFF_ANIM_OVERRIDE, nOverride, TRANSLATE_NONE ); + exportAnimPropertyuInt32( rStrm, DFF_ANIM_OVERRIDE, nOverride ); } if ( pAny[ DFF_ANIM_MASTERREL ] ) { sal_Int32 nMasterRel = 0; if ( *pAny[ DFF_ANIM_MASTERREL ] >>= nMasterRel ) - exportAnimPropertyuInt32( rStrm, DFF_ANIM_MASTERREL, nMasterRel, TRANSLATE_NONE ); + exportAnimPropertyuInt32( rStrm, DFF_ANIM_MASTERREL, nMasterRel ); } /* todo @@ -1130,7 +1130,7 @@ bool AnimationExporter::exportAnimProperty( SvStream& rStrm, const sal_uInt16 nP sal_Int32 nVal = 0; if ( rAny >>= nVal ) { - exportAnimPropertyuInt32( rStrm, nPropertyId, nVal, eTranslateMode ); + exportAnimPropertyuInt32( rStrm, nPropertyId, nVal ); bRet = true; } } @@ -1198,7 +1198,7 @@ void AnimationExporter::exportAnimPropertyFloat( SvStream& rStrm, const sal_uInt .WriteFloat( fFloat ); } -void AnimationExporter::exportAnimPropertyuInt32( SvStream& rStrm, const sal_uInt16 nPropertyId, const sal_uInt32 nVal, const TranslateMode ) +void AnimationExporter::exportAnimPropertyuInt32( SvStream& rStrm, const sal_uInt16 nPropertyId, const sal_uInt32 nVal ) { EscherExAtom aExAtom( rStrm, DFF_msofbtAnimAttributeValue, nPropertyId ); rStrm.WriteUChar( DFF_ANIM_PROP_TYPE_INT32 ) @@ -1690,11 +1690,11 @@ void AnimationExporter::exportAnimateTarget( SvStream& rStrm, const Reference< X if( nAfterEffectType != AFTEREFFECT_NONE ) { EscherExContainer aAnimPropertySet( rStrm, DFF_msofbtAnimPropertySet ); - exportAnimPropertyuInt32( rStrm, 6, 1, TRANSLATE_NONE ); + exportAnimPropertyuInt32( rStrm, 6, 1 ); if( nAfterEffectType == AFTEREFFECT_COLOR ) { - exportAnimPropertyuInt32( rStrm, 4, 0, TRANSLATE_NONE ); - exportAnimPropertyuInt32( rStrm, 5, 0, TRANSLATE_NONE ); + exportAnimPropertyuInt32( rStrm, 4, 0 ); + exportAnimPropertyuInt32( rStrm, 5, 0 ); } } exportAnimateTargetElement( rStrm, aTarget.hasValue() ? aTarget : xAnimate->getTarget(), false ); diff --git a/sd/source/filter/eppt/pptexanimations.hxx b/sd/source/filter/eppt/pptexanimations.hxx index 4628a104ac96..05583164b2d5 100644 --- a/sd/source/filter/eppt/pptexanimations.hxx +++ b/sd/source/filter/eppt/pptexanimations.hxx @@ -72,7 +72,7 @@ class AnimationExporter static bool exportAnimProperty( SvStream& rStrm, const sal_uInt16 nPropertyId, const css::uno::Any& rAny, const TranslateMode eTranslateMode ); static void exportAnimPropertyString( SvStream& rStrm, const sal_uInt16 nPropertyId, const OUString& rVal, const TranslateMode eTranslateMode ); static void exportAnimPropertyFloat( SvStream& rStrm, const sal_uInt16 nPropertyId, const double& rVal ); - static void exportAnimPropertyuInt32( SvStream& rStrm, const sal_uInt16 nPropertyId, const sal_uInt32 nVal, const TranslateMode eTranslateMode ); + static void exportAnimPropertyuInt32( SvStream& rStrm, const sal_uInt16 nPropertyId, const sal_uInt32 nVal ); static void exportAnimPropertyByte( SvStream& rStrm, const sal_uInt16 nPropertyId, const sal_uInt8 nVal ); /** if available exportAnimPropertySet diff --git a/sd/source/ui/framework/module/ResourceManager.cxx b/sd/source/ui/framework/module/ResourceManager.cxx index 6ca871a0d823..6386fc562013 100644 --- a/sd/source/ui/framework/module/ResourceManager.cxx +++ b/sd/source/ui/framework/module/ResourceManager.cxx @@ -132,7 +132,6 @@ void SAL_CALL ResourceManager::notifyConfigurationChange ( // resource managed by this ResourceManager accordingly. HandleMainViewSwitch( rEvent.ResourceId->getResourceURL(), - rEvent.Configuration, true); } } @@ -150,7 +149,6 @@ void SAL_CALL ResourceManager::notifyConfigurationChange ( { HandleMainViewSwitch( OUString(), - rEvent.Configuration, false); } else if (rEvent.ResourceId->compareTo(mxResourceId) == 0) @@ -166,7 +164,6 @@ void SAL_CALL ResourceManager::notifyConfigurationChange ( void ResourceManager::HandleMainViewSwitch ( const OUString& rsViewURL, - const Reference<XConfiguration>& /*rxConfiguration*/, const bool bIsActivated) { if (bIsActivated) diff --git a/sd/source/ui/framework/module/ResourceManager.hxx b/sd/source/ui/framework/module/ResourceManager.hxx index 080ac90d18e3..ba0d45ffcf16 100644 --- a/sd/source/ui/framework/module/ResourceManager.hxx +++ b/sd/source/ui/framework/module/ResourceManager.hxx @@ -86,7 +86,6 @@ private: void HandleMainViewSwitch ( const OUString& rsViewURL, - const css::uno::Reference<css::drawing::framework::XConfiguration>& rxConfiguration, const bool bIsActivated); void HandleResourceRequest( bool bActivation, diff --git a/sd/source/ui/sidebar/PanelFactory.cxx b/sd/source/ui/sidebar/PanelFactory.cxx index 7e1048114eea..1e05d78ec5e4 100644 --- a/sd/source/ui/sidebar/PanelFactory.cxx +++ b/sd/source/ui/sidebar/PanelFactory.cxx @@ -49,8 +49,7 @@ static Reference<lang::XEventListener> mxControllerDisposeListener; //----- PanelFactory -------------------------------------------------------- -PanelFactory::PanelFactory( - const css::uno::Reference<css::uno::XComponentContext>& /*rxContext*/) +PanelFactory::PanelFactory() : PanelFactoryInterfaceBase(m_aMutex) { } @@ -144,10 +143,10 @@ Reference<ui::XUIElement> SAL_CALL PanelFactory::createUIElement ( extern "C" SAL_DLLPUBLIC_EXPORT css::uno::XInterface* SAL_CALL -org_openoffice_comp_Draw_framework_PanelFactory_get_implementation(css::uno::XComponentContext* context, +org_openoffice_comp_Draw_framework_PanelFactory_get_implementation(css::uno::XComponentContext* /*context*/, css::uno::Sequence<css::uno::Any> const &) { - return cppu::acquire(new sd::sidebar::PanelFactory(context)); + return cppu::acquire(new sd::sidebar::PanelFactory); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/sd/source/ui/sidebar/PanelFactory.hxx b/sd/source/ui/sidebar/PanelFactory.hxx index 444438255ea0..921c07e2fac6 100644 --- a/sd/source/ui/sidebar/PanelFactory.hxx +++ b/sd/source/ui/sidebar/PanelFactory.hxx @@ -46,7 +46,7 @@ class PanelFactory public PanelFactoryInterfaceBase { public: - explicit PanelFactory (const css::uno::Reference<css::uno::XComponentContext>& rxContext); + explicit PanelFactory (); virtual ~PanelFactory() override; PanelFactory(const PanelFactory&) = delete; PanelFactory& operator=(const PanelFactory&) = delete; |