Re: [Libreoffice] [PUSHED] [PATCH] Remove dead code

Nice, thanks :) Pushed it.

Thomas

On 02/28/2011 11:21 PM, Michael Lefevre wrote:
> ACCESSIBLE_EVENT_NOTIFICATION_ENABLED was never define. So the #ifdef #endif "brackets" could go 
> away.
> ---
>  .../extended/accessibleiconchoicectrlentry.hxx     |   16 ----------------
>  .../extended/accessibleiconchoicectrlentry.cxx     |   15 ---------------
>  2 files changed, 0 insertions(+), 31 deletions(-)
>
> diff --git a/accessibility/inc/accessibility/extended/accessibleiconchoicectrlentry.hxx 
> b/accessibility/inc/accessibility/extended/accessibleiconchoicectrlentry.hxx
> index 3e9cec0..b2c145d 100644
> --- a/accessibility/inc/accessibility/extended/accessibleiconchoicectrlentry.hxx
> +++ b/accessibility/inc/accessibility/extended/accessibleiconchoicectrlentry.hxx
> @@ -83,22 +83,6 @@ namespace accessibility
>          ::com::sun::star::uno::Reference< ::com::sun::star::accessibility::XAccessible > 
> m_xParent;
>  
>      private:
> -    #ifdef ACCESSIBLE_EVENT_NOTIFICATION_ENABLED
> -    // (the following method is unused currently. If you need it, simply remove the #ifdef thing 
> here and
> -    // in the cxx)
> -        /** notifies all listeners that this object has changed
> -            @param   _nEventId
> -                is the event id
> -            @param   _aOldValue
> -                is the old value
> -            @param   _aNewValue
> -                is the new value
> -        */
> -        void NotifyAccessibleEvent( sal_Int16 _nEventId,
> -                                       const ::com::sun::star::uno::Any& _aOldValue,
> -                                       const ::com::sun::star::uno::Any& _aNewValue );
> -    #endif
> -
>          Rectangle                            GetBoundingBox_Impl() const;
>          Rectangle                            GetBoundingBoxOnScreen_Impl() const;
>          sal_Bool                             IsAlive_Impl() const;
> diff --git a/accessibility/source/extended/accessibleiconchoicectrlentry.cxx 
> b/accessibility/source/extended/accessibleiconchoicectrlentry.cxx
> index 6a613de..791d89a 100644
> --- a/accessibility/source/extended/accessibleiconchoicectrlentry.cxx
> +++ b/accessibility/source/extended/accessibleiconchoicectrlentry.cxx
> @@ -122,21 +122,6 @@ throw(RuntimeException)
>              dispose();
>          }
>      }
> -    #ifdef ACCESSIBLE_EVENT_NOTIFICATION_ENABLED
> -    // (the following method is unused currently. If you need it, simply remove the #ifdef thing 
> here and
> -    // in the hxx)
> -    // -----------------------------------------------------------------------------
> -    void AccessibleIconChoiceCtrlEntry::NotifyAccessibleEvent( sal_Int16 _nEventId,
> -                                                   const ::com::sun::star::uno::Any& _aOldValue,
> -                                                   const ::com::sun::star::uno::Any& _aNewValue )
> -    {
> -        Reference< uno::XInterface > xSource( *this );
> -        AccessibleEventObject aEventObj( xSource, _nEventId, _aNewValue, _aOldValue );
> -
> -        if (m_nClientId)
> -            comphelper::AccessibleEventNotifier::addEvent( m_nClientId, aEventObj );
> -    }
> -    #endif
>      // -----------------------------------------------------------------------------
>      Rectangle AccessibleIconChoiceCtrlEntry::GetBoundingBox_Impl() const
>      {