[PUSHED][PATCH] Remove unused code

Caolán McNamara caolanm at redhat.com
Tue Mar 20 06:54:36 PDT 2012


On Sat, 2012-03-17 at 15:34 +0100, Chr. Rossmanith wrote:
> Hi,
> 
> I had a look at unusedcode.easy (sdext/presenter) and removed a little 
> bit more than mentioned there: as there are no end callbacks added 
> (AddEndCallback() was unused), there is no need to run them -> 
> RunEndCallbacks() could be removed as well. Same logic for start 
> callbacks. Finally mpStartCallbacks and mpEndCallbacks could leave, too.

Yup, I agree, I pushed this now.

> In PresenterAnimator AddAnimation() was unused and it seems that the 
> whole class might be removed?!?

Yeah, in unused.easy I see that the PresenterAnimator::PresenterAnimator
ctor is never called, and its the only constructor, so one is never
instantiated, so you can remove the whole thing without loss of
functionality.

C.



More information about the LibreOffice mailing list