2011/6/27 Radek Doulík <r...@novell.com>

> Hi Luke,
>
> > @@ -542,6 +543,7 @@ void
> > CustomAnimationList::select( CustomAnimationEffectPtr pEffect, bool
> > bSelect
> >      {
> >          append( pEffect );
> >          select( pEffect );
> > +        MakeVisible( Last() );
>
> we don't need to call it here ^^^, as select( pEffect ); on the previous
> line recursively calls this function again and it is made visible from
> the loop above already. So it should be enough to add it to the loop.
>

Ah yes, of course!

I have updated your patch and pushed it.


Thanks very much :)

>
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to