Re: RFR: JDK-8030179: java/nio/Buffer/Chars.java, testcases seems all pass but jtreg/testng failed with java.lang.AssertionError

2015-02-07 Thread Paul Sandoz
On Feb 6, 2015, at 8:40 PM, Xueming Shen xueming.s...@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

Re: RFR: JDK-8068682 - Deprivilege/move java.corba to the ext class loader

2015-02-07 Thread David Holmes
On 7/02/2015 4:55 AM, Mark Sheppard wrote: Hi please oblige and review the following changes http://cr.openjdk.java.net/~msheppar/8068682/webrev/ http://cr.openjdk.java.net/~msheppar/8068682/corba/webrev/ which address the issue in https://bugs.openjdk.java.net/browse/JDK-8068682 this

Re: RFR: JDK-8068682 - Deprivilege/move java.corba to the ext class loader

2015-02-07 Thread Mandy Chung
On 2/7/2015 5:10 PM, David Holmes wrote: On 7/02/2015 4:55 AM, Mark Sheppard wrote: Hi please oblige and review the following changes http://cr.openjdk.java.net/~msheppar/8068682/webrev/ http://cr.openjdk.java.net/~msheppar/8068682/corba/webrev/ which address the issue in