Re[2]: [Gnash-commit] gnash ChangeLog server/parser/shape_character_d...

[Udo Giacomozzi]

Hello strk,

Monday, November 12, 2007, 3:39:21 PM, you wrote:
s> Gnash is giving a false positive since your change.

Right. A new commit is on the way that fixes this...

s> Could be the cirle fill (defined by curves).

It is the blue shape (red outline) on the left side. It fails because
the test coordinate is at the very top of that shape and that's where
the current implementation fails. But I've fixed this now and will
commit it in a few moments (running test suite ATM).

BTW, it's a bit of a problem having so much shapes on the stage at the
same time and doing hit tests. It can produce false positives and
negatives (happened at least twice) - which is good and bad at the
same time. Good because otherwise we probably wouldn't have noticed
this last bug and bad because it perhaps hides other bugs.

A general word to test cases: It's pratically *impossible* to create
test cases for every situation. While working on the point_test
implementation I found an endless number of situations, each would
need a dozen test points...


Udo