Hi Joe, I think this looks OK. My only suggestion is I am not a fan of creating a one-off type of test vs integrating if at all possible a test such as this into existing test(s). The reason is it can get out of control the number of one off tests and makes it a bit harder to know the coverage if you have a slew of files to review.
Again, feel free to push, and ignore me :-) Best Lance On Dec 11, 2015, at 12:54 PM, huizhe wang <huizhe.w...@oracle.com> wrote: > This is a fix for an edge case. All tests passed after the fix. > > JBS: > https://bugs.openjdk.java.net/browse/JDK-8068839 > > webrev: > http://cr.openjdk.java.net/~joehw/jdk9/8068839/webrev/ > > Thanks, > Joe > Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com
