RFR: JDK-8030179: java/nio/Buffer/Chars.java, testcases seems all pass but jtreg/testng failed with java.lang.AssertionError
Paul Sandoz
paul.sandoz at oracle.com
Sat Feb 7 15:39:05 UTC 2015
On Feb 6, 2015, at 8:40 PM, Xueming Shen <xueming.shen at oracle.com> wrote:
> Thanks Paul!
>
> Webrev has been updated to include your test, and to fix that stackoverflowexception, triggered
> by the recursive implementation of StreamEncoder.flushLeftoverChar. The method really does
> not need to be recursive (an alternative is to push the "leftover" from the input buffer back to
> the buffer, but it appears the proposed change is simple)
>
> http://cr.openjdk.java.net/~sherman/8030179/webrev/
>
+1
Paul.
More information about the core-libs-dev
mailing list