[PUSHED][PATCH] Remove unused code
Chr. Rossmanith
ChrRossmanith at gmx.de
Tue Mar 20 13:26:33 PDT 2012
Am 20.03.2012 15:56, schrieb Caolán McNamara:
> On Tue, 2012-03-20 at 13:54 +0000, Caolán McNamara wrote:
>> 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.
> Whoops, its PresenterAnimation (not PresenterAnimator) which has only
> one ctor that is never used and so is never instantiated and is trivial
> to remove.
A PresenterAnimation could be instantiated here
ModeChangeAnimation (
const ::boost::shared_ptr<PresenterSprite>& rpSprite,
const Reference<rendering::XSpriteCanvas>& rxCanvas)
: PresenterAnimation (0, 1000, 20),
mpSprite(rpSprite),
mxCanvas(rxCanvas)
{
}
in PresenterWindowManager.cxx - iff ModeChangeAnimation would be used
anywhere. I can't find any place where it is used. Double check would be
nice. So first ModeChangeAnimation would have to go, then
PresenterAnimation could be removed as well.
Christina
More information about the LibreOffice
mailing list