Re: Making java.util.Iterator.remove() for the iterators for EnumSet more resilient

2011-03-03 Thread Neil Richards
Please advise me on what steps remain for this change to be committed. Thanks, Neil -- Unless stated above: IBM email: neil_richards at uk.ibm.com IBM United Kingdom Limited - Registered in England and Wales with number 741598. Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire

hg: jdk7/tl/langtools: 7023233: False positive for -Xlint:try with nested try with resources blocks

2011-03-03 Thread maurizio . cimadamore
Changeset: e9b8fbb30f5a Author:mcimadamore Date: 2011-03-03 09:43 + URL: http://hg.openjdk.java.net/jdk7/tl/langtools/rev/e9b8fbb30f5a 7023233: False positive for -Xlint:try with nested try with resources blocks Summary: Wrong lint warning issued about unused resource when

hg: jdk7/tl/jaxp: 7023289: jaxp 1.4.5 development jdk7 2nd integration

2011-03-03 Thread kelly . ohair
Changeset: 877fd25c5a2f Author:ohair Date: 2011-03-02 12:00 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/jaxp/rev/877fd25c5a2f 7023289: jaxp 1.4.5 development jdk7 2nd integration Reviewed-by: joehw, mchung, alanb ! jaxp.properties

Re: Making java.util.Iterator.remove() for the iterators for EnumSet more resilient

2011-03-03 Thread Alan Bateman
Neil Richards wrote: Please advise me on what steps remain for this change to be committed. I think the main thing with this proposal is getting agreement that it is the right thing to do. On one hand it avoids the set getting corrupted. On the other hand it's masking a problem and really

hg: jdk7/tl/jdk: 7018606: (process) test/java/lang/ProcessBuilder/Basic.java failing intermittently (win)

2011-03-03 Thread michael . x . mcmahon
Changeset: 75064373ed6b Author:michaelm Date: 2011-03-03 15:34 + URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/75064373ed6b 7018606: (process) test/java/lang/ProcessBuilder/Basic.java failing intermittently (win) Reviewed-by: alanb ! test/java/lang/ProcessBuilder/Basic.java

Re: Making java.util.Iterator.remove() for the iterators for EnumSet more resilient

2011-03-03 Thread Neil Richards
On 3 March 2011 15:06, Alan Bateman alan.bate...@oracle.com wrote: I think the main thing with this proposal is getting agreement that it is the right thing to do. On one hand it avoids the set getting corrupted. On the other hand it's masking a problem and really part of a bigger issue. This

hg: jdk7/tl/jdk: 7018137: HTML4 compliance issues

2011-03-03 Thread chris . hegarty
Changeset: 7cfb0693eb33 Author:chegar Date: 2011-03-03 16:44 + URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/7cfb0693eb33 7018137: HTML4 compliance issues Summary: move end list items tags to after nested list Reviewed-by: alanb ! src/share/classes/java/net/URI.java !

hg: jdk7/tl/jdk: 6721694: java/lang/Thread/StartOOMTest.java fails with timeout or with crash

2011-03-03 Thread chris . hegarty
Changeset: 9b99a14375bc Author:chegar Date: 2011-03-03 16:48 + URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/9b99a14375bc 6721694: java/lang/Thread/StartOOMTest.java fails with timeout or with crash Summary: the test is not suitable to be run automatically Reviewed-by: alanb

hg: jdk7/tl/jdk: 2 new changesets

2011-03-03 Thread sean . coffey
Changeset: 732faed56eb0 Author:coffeys Date: 2011-03-03 16:51 + URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/732faed56eb0 6750362: Very large LDAP requests throw a OOM on LDAP servers which aren't aware of Paged Results Controls 6748156: add an new JNDI property to control

hg: jdk7/tl/jdk: 2 new changesets

2011-03-03 Thread michael . x . mcmahon
Changeset: b645b5bc460b Author:michaelm Date: 2011-03-03 17:14 + URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/b645b5bc460b 7024264: HttpURLConnection/NetPermission doc issue Reviewed-by: chegar ! src/share/classes/java/net/HttpURLConnection.java !

hg: jdk7/tl/langtools: 2 new changesets

2011-03-03 Thread maurizio . cimadamore
Changeset: c15d788cb381 Author:mcimadamore Date: 2011-03-03 17:32 + URL: http://hg.openjdk.java.net/jdk7/tl/langtools/rev/c15d788cb381 7023703: Valid code doesn't compile Summary: leftovers cause problems when analyzing loops in Flow.java Reviewed-by: jjg !

hg: jdk7/tl/langtools: 7024212: TestWarnErrorCount fails

2011-03-03 Thread maurizio . cimadamore
Changeset: 8fb48a9ac9ec Author:mcimadamore Date: 2011-03-03 18:05 + URL: http://hg.openjdk.java.net/jdk7/tl/langtools/rev/8fb48a9ac9ec 7024212: TestWarnErrorCount fails Summary: TestWarnErrorCount should be executed with -Xlint:all,-path to avoid spurious failures Reviewed-by:

7023963 : Simple fixes to anon diamond in the jdk.

2011-03-03 Thread Kumar Srinivasan
Please review, corrections to the diamond operator on anonymous classes as suggested by langtools team. http://cr.openjdk.java.net/~ksrini/7023963/webrev.0/ Thanks Kumar

Re: 7023963 : Simple fixes to anon diamond in the jdk.

2011-03-03 Thread Mandy Chung
Looks fine with me. Mandy On 03/03/11 11:40, Kumar Srinivasan wrote: Please review, corrections to the diamond operator on anonymous classes as suggested by langtools team. http://cr.openjdk.java.net/~ksrini/7023963/webrev.0/ Thanks Kumar

Re: 7023963 : Simple fixes to anon diamond in the jdk.

2011-03-03 Thread David Holmes
Kumar Srinivasan said the following on 03/04/11 05:40: Please review, corrections to the diamond operator on anonymous classes as suggested by langtools team. http://cr.openjdk.java.net/~ksrini/7023963/webrev.0/ Looks fine of course, but it is sad that it is necessary. I wonder what

Re: 7023963 : Simple fixes to anon diamond in the jdk.

2011-03-03 Thread maurizio cimadamore
On 03/03/2011 20:43, David Holmes wrote: Kumar Srinivasan said the following on 03/04/11 05:40: Please review, corrections to the diamond operator on anonymous classes as suggested by langtools team. http://cr.openjdk.java.net/~ksrini/7023963/webrev.0/ Looks fine of course, but it is sad

Request for review: 7022370 Launcher ergonomics doesn't need per-architecture implementations

2011-03-03 Thread David Holmes
Hopefully all interested parties are addressed in the cc lists. webrev at: http://cr.openjdk.java.net/~dholmes/7022370/webrev/ The launcher ergonomics (ergo.c) currently relies on per-architecture, eg ergo_sparc.c, ergo_i586.c, files to define the actual ergonomics operations. Only x86 is