http://cr.openjdk.java.net/~prr/8052396.1 adds a break to keep findbugs
happy.
2nd reviewer anyone ?
-phil.
On 07/28/2014 11:28 PM, Andrew Brygin wrote:
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.
