gnash-commit
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Gnash-commit] gnash ChangeLog server/character.cpp server/cha...


From: Sandro Santilli
Subject: Re: [Gnash-commit] gnash ChangeLog server/character.cpp server/cha...
Date: Tue, 4 Sep 2007 09:02:09 +0200

On Tue, Sep 04, 2007 at 02:57:35PM +0800, zou lunkai wrote:
> > +       std::for_each(toReinsert.begin(), toReinsert.end(),
> > +               boost::bind(&DisplayList::reinsertRemovedCharacter, this, 
> > _1));
> > +
> I understand this is still under working. Just for a discussion.
> Is it too expensive to re-insert all removed characters? I think
> re-insertion is only needed for sprites. AFAIK, no other characters
> support onUnload/onLoad.

toReinsert should only contain characters whose ::unload() method
returned true, which is supposed to return true only if they have
unload methods to be invoked. The ::unload() method itself (in ::character)
might be getting more characters then needed as it doesn't consider
the type of character, just the availability of onUnload events
(both clip and user-defined). You may try setting onUnload for shape
or morphs, or TextFields too and see what happens.

--strk;




reply via email to

[Prev in Thread] Current Thread [Next in Thread]