From 1f43eab48a0397d9f7c272116aeb150694b1721d Mon Sep 17 00:00:00 2001 From: Stephan Bergmann <sbergman@redhat.com> Date: Sun, 10 May 2015 15:59:31 +0200 Subject: SfxMacroTabPage::mpImpl is never assigned to... ...so these must be redundant aliases Change-Id: I6c48f24f7808f336e206d8cba257f9bc063bdfc7 --- cui/source/tabpages/macroass.cxx | 21 +++++++++------------ 1 file changed, 9 insertions(+), 12 deletions(-) diff --git a/cui/source/tabpages/macroass.cxx b/cui/source/tabpages/macroass.cxx index b88272223efe..2e90ed844306 100644 --- a/cui/source/tabpages/macroass.cxx +++ b/cui/source/tabpages/macroass.cxx @@ -263,8 +263,7 @@ bool _SfxMacroTabPage::IsReadOnly() const IMPL_LINK( _SfxMacroTabPage, SelectEvent_Impl, SvTabListBox*, ) { - _SfxMacroTabPage_Impl* pImpl = mpImpl; - SvHeaderTabListBox& rListBox = pImpl->pEventLB->GetListBox(); + SvHeaderTabListBox& rListBox = mpImpl->pEventLB->GetListBox(); SvTreeListEntry* pE = rListBox.FirstSelected(); sal_uLong nPos; if( !pE || LISTBOX_ENTRY_NOTFOUND == @@ -281,13 +280,12 @@ IMPL_LINK( _SfxMacroTabPage, SelectEvent_Impl, SvTabListBox*, ) IMPL_LINK( _SfxMacroTabPage, SelectGroup_Impl, ListBox*, ) { - _SfxMacroTabPage_Impl* pImpl = mpImpl; - pImpl->pGroupLB->GroupSelected(); - const OUString sScriptURI = pImpl->pMacroLB->GetSelectedScriptURI(); + mpImpl->pGroupLB->GroupSelected(); + const OUString sScriptURI = mpImpl->pMacroLB->GetSelectedScriptURI(); OUString aLabelText; if( !sScriptURI.isEmpty() ) - aLabelText = pImpl->maStaticMacroLBLabel; - pImpl->pMacroFrame->set_label( aLabelText ); + aLabelText = mpImpl->maStaticMacroLBLabel; + mpImpl->pMacroFrame->set_label( aLabelText ); EnableButtons(); return 0; @@ -301,8 +299,7 @@ IMPL_LINK( _SfxMacroTabPage, SelectMacro_Impl, ListBox*, ) IMPL_LINK( _SfxMacroTabPage, AssignDeleteHdl_Impl, PushButton*, pBtn ) { - _SfxMacroTabPage_Impl* pImpl = mpImpl; - SvHeaderTabListBox& rListBox = pImpl->pEventLB->GetListBox(); + SvHeaderTabListBox& rListBox = mpImpl->pEventLB->GetListBox(); SvTreeListEntry* pE = rListBox.FirstSelected(); sal_uLong nPos; if( !pE || LISTBOX_ENTRY_NOTFOUND == @@ -312,7 +309,7 @@ IMPL_LINK( _SfxMacroTabPage, AssignDeleteHdl_Impl, PushButton*, pBtn ) return 0; } - const bool bAssEnabled = pBtn != pImpl->pDeletePB && pImpl->pAssignPB->IsEnabled(); + const bool bAssEnabled = pBtn != mpImpl->pDeletePB && mpImpl->pAssignPB->IsEnabled(); // remove from the table sal_uInt16 nEvent = (sal_uInt16)reinterpret_cast<sal_uLong>(pE->GetUserData()); @@ -321,7 +318,7 @@ IMPL_LINK( _SfxMacroTabPage, AssignDeleteHdl_Impl, PushButton*, pBtn ) OUString sScriptURI; if( bAssEnabled ) { - sScriptURI = pImpl->pMacroLB->GetSelectedScriptURI(); + sScriptURI = mpImpl->pMacroLB->GetSelectedScriptURI(); if( sScriptURI.startsWith( "vnd.sun.star.script:" ) ) { aTbl.Insert( @@ -335,7 +332,7 @@ IMPL_LINK( _SfxMacroTabPage, AssignDeleteHdl_Impl, PushButton*, pBtn ) } } - pImpl->pEventLB->SetUpdateMode( false ); + mpImpl->pEventLB->SetUpdateMode( false ); pE->ReplaceItem( new SvLBoxString( pE, 0, sScriptURI ), LB_MACROS_ITEMPOS ); rListBox.GetModel()->InvalidateEntry( pE ); rListBox.Select( pE ); -- cgit