That's a lot of test case for one path, but it does indeed fail before and pass after the fix so it looks like it does the job.

+1 from me.

                        ...jim

On 12/15/15 2:06 PM, Laurent Bourgès wrote:
Jim & Phil,

Here is an updated webrev:
http://cr.openjdk.java.net/~lbourges/marlin/Stroker-8144718.3/

I have totally rewritten the test to:
- enable Marlin doChecks=true
- use Marlin logger + custom handler to detect any failure in
XxxArrayCache.check()
- serialize the 'bad' shape into java code: see serialize(shape) and
derserialize()

In conclusion, the improved test fails without the fix and no more need
any font (embedded shape in the java code).

Cheers,
Laurent

Reply via email to