Re[2]: [Gnash-commit] gnash ChangeLog server/character.h server/dlist...

[Udo Giacomozzi]

Hello Sandro,

Tuesday, October 30, 2007, 10:37:39 AM, you wrote:
SS> If you remember I asked you "do you have setMask in mind" when you
SS> added the new intelligence.

Yes, my answer was: I don't know much about setMask()  :)


SS> Maybe time to prove it against setMask now ?

Sure, but you'll have to wait (lack of time)  :(


SS> Note that dynamic masks (set by setMask) do not have to be in the
SS> same DisplayList at all ! (but I belive we don't have an automated
SS> test for this yet) 

Uhm... that may be a problem. But in any case the
add_invalidated_bounds() method is strongly related to the display()
method, i.e. instead of calling renderer mask methods and display() of
childs, it calls add_invalidated_bounds() of the childs and handles
the masks internally.

So, any changes to display() should also be replicated to
add_invalidated_bounds(). Hmm, might be worth to add a comment about
this relationship in both methods.


Udo