The change looks ok to me Sherman. Should the bug synopsis be updated? It looks like a test bug rather than a product one.
-Chris > On 5 Aug 2015, at 20:04, Xueming Shen <[email protected]> wrote: > > Hi, > > Would you please help review the proposed change for JDK-8022224. > > issue: https://bugs.openjdk.java.net/browse/JDK-8022224 > webrev: http://cr.openjdk.java.net/~sherman/8022224/ > > The fix for this particular problem is at ln#135-#137. The rest is the > clean to remove unused the code (was for the sun.nio.cs impl, which > had been completely removed since jdk7). > > Thanks, > -Sherman
