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

[Sandro Santilli]

On Tue, Oct 30, 2007 at 09:26:50AM +0100, Udo Giacomozzi wrote:
> Hello Sandro,
> 
> Monday, October 29, 2007, 10:23:16 PM, you wrote:
> SS> * server/character.h (setMask): set character as invalidated
> SS>   when it's mask change. Needs more care: when the mask itself
> SS>   changes, we should find a way to invalidate the maskee as well,
> SS>   but we currently have no refefence from the mask to the maskee.
> 
> I haven't checked your changes yet (just reading ChangeLog) but
> invalidating the maskee itself should not be necessary. The
> invalidation stuff is only about finding the stage *area* that has to
> be re-rendered.
> 
> The mask and the maskee by nature are on "top of each other". So, when
> invalidating the mask you implicitely force also re-rendering of the
> maskee.

Yes, makes sense, thanks.
"top of each other" is not necessarely true though....
if the mask and the maskee do not overlap moving the mask out of the
maskee bounding box  shouldn't make the maskee invalidated.

Maybe more then talking about invalidated *state* the TODO is about
the invalidated *bounds*.
Like, get_invalidated_bounds for a masked object might only returns
the invalidated bounds resulting from intersection of own 
invalidated bounds and maks's invalidated bounds.

--strk;