Hello Phil, in general, the change looks fine to me.
The only concern is whether the code at FileFontStrike.java lines 498 - 506
triggers tools like findbug due to missed 'break' statement. Does it worth to add 'break' there? Thanks, Andrew On 7/29/2014 2:30 AM, Phil Race wrote:
https://bugs.openjdk.java.net/browse/JDK-8052396 http://cr.openjdk.java.net/~prr/8052396/ This is a robustness fix. Nothing changes unless we get exceptions indexing glyphs. This has been seen only in a stress test at the same time as exiting the VM.
