[OpenJDK 2D-Dev] RFR 8144938: Handle properly coordinate overflow in Marlin Renderer
Laurent Bourgès
bourges.laurent at gmail.com
Fri Mar 18 16:52:36 UTC 2016
Jim,
Here is a final webrev (for archive):
http://cr.openjdk.java.net/~lbourges/marlin/marlin-8144938.5/
Changes:
- remove loops in CrashNaNTest
Hope it is ready to go !
I need another reviewer, please.
2016-03-18 1:44 GMT+01:00 Jim Graham <james.graham at oracle.com>:
> Hi Laurent,
>
> That looks fine. If the subPathStarted changes are the only changes then
> it's good to go. If there was some other change that I missed, let me know
> so I can review it as well.
>
No this webrev had only changes in MRE.pathToLoop() you reviewed.
>
> One curiosity - in the test case, why are there 3 for loops that perform
> exactly one pass? (lines 134, 221, 282) I'm guessing that maybe in a
> former life this code may have been part of a performance suite or
> something? It's not a bug, just an odd bit of code. You can delete them
> if you want, but I don't need to see a review of that...
>
Fixed, now.
Cheers,
Laurent
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20160318/5c4343ad/attachment.html>
More information about the 2d-dev
mailing list