Re: RFR: 8062194: java.util.jar.Attributes should use insertion-ordered iteration

2014-11-07 Thread Alan Bateman
On 05/11/2014 00:06, Martin Buchholz wrote: : So ... there is the question whether we are tightening the spec for Attributes and its subclasses or only for Attributes. Subclasses of Attribute that would change the order are likely to be very rare (is there a use case?) - I'm OK with those becomin

Re: [9] Review request : JDK-8059070: [TESTBUG] java/lang/invoke/LFCaching/LFMultiThreadCachingTest.java failed - timeout

2014-11-07 Thread Konstantin Shefov
Gently reminder 29.10.2014 17:25, Konstantin Shefov пишет: Please, review a test bug fix. http://cr.openjdk.java.net/~kshefov/8059070/webrev.01/ -Konstantin On 27.10.2014 13:16, Konstantin Shefov wrote: Kindly reminder On 23.10.2014 19:04, Paul Sandoz wrote: On Oct 23, 2014, at 1:25 PM, Kon

Re: [9] Review request : JDK-8062904: TEST_BUG: Tests java/lang/invoke/LFCaching fail when run with -Xcomp option

2014-11-07 Thread Konstantin Shefov
Kindly reminder http://cr.openjdk.java.net/~kshefov/8062904/webrev.01/ 06.11.2014 18:20, Konstantin Shefov пишет: Igor, Thanks for your comment! Here is the updated webrev: http://cr.openjdk.java.net/~kshefov/8062904/webrev.01/ -Konstantin On 06.11.2014 17:36, Igor Ignatyev wrote: Konstan

Re: [9] Review request : JDK-8062904: TEST_BUG: Tests java/lang/invoke/LFCaching fail when run with -Xcomp option

2014-11-07 Thread Igor Ignatyev
Ho Konstantin, Actually, I don't think that this should be fixed by reducing iterations, CompileCommand magic sounds more reasonable here. I'd even say I'm not sure that it's a test bug, it looks very similar to JDK-8046809. have you tried to run the tests on the build where JDK-8046809 is f

Re: RFR: 8060132: Handlers configured on abstract nodes in logging.properties are not always properly closed

2014-11-07 Thread Daniel Fuchs
On 05/11/14 19:47, Stanimir Simeonoff wrote: 945 } catch (Exception ex) { 946 System.err.println("Can't load log handler \"" + type + "\""); 947 System.err.println("" + ex); 948 ex.printStackTrace

Re: RFR: 8060132: Handlers configured on abstract nodes in logging.properties are not always properly closed

2014-11-07 Thread Daniel Fuchs
On 07/11/14 07:42, Mandy Chung wrote: On 11/5/2014 8:15 AM, Daniel Fuchs wrote: Here is the new webrev: http://cr.openjdk.java.net/~dfuchs/webrev_8060132/webrev.03/index.html Looks good. Minor nits and no need to generate new webrev. line 108: {@linkplain #reset} -- # is missing. line 918-9

RFR JDK-8059311: com/sun/jndi/ldap/LdapTimeoutTest.java fails with exit_code == 0

2014-11-07 Thread Pavel Rappo
Hi everyone, Could you please review my change for JDK-8059311? http://cr.openjdk.java.net/~prappo/8059311/webrev.00/ It looks like one of the "killSwitches" went off and executed System.exit(0). The problem is jtreg doesn't like

Re: Review request for 8055063: Parameter#toString() fails w/ AIOOBE for ctr of inner class w/ generic type

2014-11-07 Thread Eric McCorkle
Are there any more comments on this patch? This is an 8u40 issue, so it needs to be committed soon. Thanks, Eric On 11/05/14 16:57, Eric McCorkle wrote: > A word on the tests: I don't think there's a stable way to check the > types on getParameterizedType() and getAnnotatedType(). So I checked

Re: RFR JDK-8059311: com/sun/jndi/ldap/LdapTimeoutTest.java fails with exit_code == 0

2014-11-07 Thread Vincent Ryan
All looks good. Thanks. On 7 Nov 2014, at 12:14, Pavel Rappo wrote: > Hi everyone, > > Could you please review my change for JDK-8059311? > > http://cr.openjdk.java.net/~prappo/8059311/webrev.00/ > > > It looks like one of the

RFR 8064356: Add BaseRowSet, SQLInputImpl, and SQLOutputImpl tests

2014-11-07 Thread Lance Andersen
Hi all, Need a reviewer for the following tests for exercising BaseRowSet, SQlInputImpl, and SQLOutputImpl The webrev can be found at: http://cr.openjdk.java.net/~lancea/8064356/webrev.00/ Have a good weekend! Best, Lance Lance Andersen| Principal Member of Technical Staff | +1.781.442.203

RFR (JAXP) 8059327 : XML parser returns corrupt attribute value

2014-11-07 Thread huizhe wang
Hi, This is a regression caused by changeset: http://cr.openjdk.java.net/~joehw/jdk9/8027359/webrev/. The fix then is to recover those changes that modified the buffer. Please review. http://cr.openjdk.java.net/~joehw/jdk9/8059327/webrev/ https://bugs.openjdk.java.net/browse/JDK-8059327 Than

Re: RFR (JAXP) 8059327 : XML parser returns corrupt attribute value

2014-11-07 Thread Lance Andersen
Looks OK Joe. I went and looked at the previous version prior to the 1st fix. Best Lance On Nov 7, 2014, at 11:50 AM, huizhe wang wrote: > Hi, > > This is a regression caused by changeset: > http://cr.openjdk.java.net/~joehw/jdk9/8027359/webrev/. The fix then is to > recover those changes th

Re: RFR 8064356: Add BaseRowSet, SQLInputImpl, and SQLOutputImpl tests

2014-11-07 Thread Roger Riggs
Hi Lance, Looks ok. Roger On 11/7/14 11:25 AM, Lance Andersen wrote: Hi all, Need a reviewer for the following tests for exercising BaseRowSet, SQlInputImpl, and SQLOutputImpl The webrev can be found at: http://cr.openjdk.java.net/~lancea/8064356/webrev.00/ Have a good weekend! Best, La

Re: RFR 8064356: Add BaseRowSet, SQLInputImpl, and SQLOutputImpl tests

2014-11-07 Thread huizhe wang
+1, nice tests! Joe On 11/7/2014 9:47 AM, Roger Riggs wrote: Hi Lance, Looks ok. Roger On 11/7/14 11:25 AM, Lance Andersen wrote: Hi all, Need a reviewer for the following tests for exercising BaseRowSet, SQlInputImpl, and SQLOutputImpl The webrev can be found at: http://cr.openjdk.jav

Re: RFR: 8062194: java.util.jar.Attributes should use insertion-ordered iteration

2014-11-07 Thread Martin Buchholz
On Fri, Nov 7, 2014 at 1:52 AM, Alan Bateman wrote: > On 05/11/2014 00:06, Martin Buchholz wrote: >> >> : >> So ... there is the question whether we are tightening the spec for >> Attributes and its subclasses or only for Attributes. >> Subclasses of Attribute that would change the order are likel

Re: [9] RFR(L) 8013267 : move MemberNameTable from native code to Java heap, use to intern MemberNames

2014-11-07 Thread David Chase
New webrev: bug: https://bugs.openjdk.java.net/browse/JDK-8013267 webrevs: http://cr.openjdk.java.net/~drchase/8013267/jdk.06/ http://cr.openjdk.java.net/~drchase/8013267/hotspot.06/ Changes since last: 1) refactored to put ClassData under java.lang.invoke.MemberName 2) split the data structur

Re: RFR: 8062771: Core reflection should use final fields whenever possible

2014-11-07 Thread Joel Borggrén-Franck
Hi Martin, Thanks for the clarification. On 6 nov 2014, at 20:51, Martin Buchholz wrote: > Hi Joel, > > On Thu, Nov 6, 2014 at 2:48 AM, Joel Borggrén-Franck > wrote: >> Hi, >> >> I’m having a hard time following this thread, which webrev has been updated, >> for which release, fixing which

Re: RFR: 8062771: Core reflection should use final fields whenever possible

2014-11-07 Thread Martin Buchholz
Hi Joel, Thanks for volunteering. I foisted all I have in https://bugs.openjdk.java.net/browse/JDK-8064391 I volunteer to be your reviewer for the backports. On Fri, Nov 7, 2014 at 1:36 PM, Joel Borggrén-Franck wrote: > Hi Martin, > > Thanks for the clarification. > > On 6 nov 2014, at 20:51,