In the test case, why are you using a log handler to check for a
particular exception? Shouldn't any exception logged be cause for a
test failure?
...jim
On 2/8/16 1:22 PM, Laurent Bourgès wrote:
Phil & Jim,
Please review a simple fix for a SEGV in Marlin renderer due to NaN
coordinates:
bug: https://bugs.openjdk.java.net/browse/JDK-8149338
webrev: http://cr.openjdk.java.net/~lbourges/marlin/marlin-8149338.0/
Changes:
- addLine: use first / last Y crossings to compute edge min/max Y that
ensures consistency with edgeBuckets / edgeBucketCounts arrays in case
of NaN values: it is used in dispose() to reset these critical arrays
(pointer-like) => robust and consistent zero-fill
- minor cleanup in closePath() to avoid upscaling coordinates
- added crash test
Cheers,
Laurent
