[OpenJDK Rasterizer] [OpenJDK 2D-Dev] Path2D optimizations

[Laurent Bourgès]

Jim,

Test program, line 490 - MOVETO has 2 coordinates associated with it.
>

Well spotted: I did it too quickly, sorry.

Test program, line 492 - perhaps we should throw an exception on default
> since it indicates a problem with the iterator
>

Ok.


> Other than that, it looks good...
>

Thanks, I will tomorrow prepare a new webrev.

 - What am I missing on the difference between the two test files?  Can
>>> they be combined into a single test file?
>>>
>>
>> Big one: I figured out a bug in Path2d happening with 0 capacity:
>> rectCrossing and ptCrossing fail.
>>
>
> I'm guessing that is why there are the new shortcut returns on the
> intersects/contains methods in Path2D?  I see it now...
>

Exactly: that test fails on JDK 1.8.0_60.

Should we submit another bug to JDK8 ?
Or at least, ask to backport this bug once fixed.

Laurent
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/graphics-rasterizer-dev/attachments/20150330/5a2784cb/attachment.html>