RFR: 8022854 : (s) Cleaner re-initialization of System properties

2014-01-27 Thread Mike Duigou
Hello all; This is a bit of cleanup I did back during Java 8 that got deferred due to it's late arrival during the development cycle. I've updated it for Java 9. http://cr.openjdk.java.net/~mduigou/JDK-8022854/0/webrev/ This change improves the implementation of System.setProperties(null) which

Re: RFR: 8022854 : (s) Cleaner re-initialization of System properties

2014-01-28 Thread Paul Sandoz
Hi Mike, Your patch is slightly out of sync with 9: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/c8c4f441fc76 http://hg.openjdk.java.net/jdk9/dev/jdk/file/tip/src/share/classes/sun/misc/VM.java -- Generally looks OK. Bikeshed-wise i prefer something like getPublicSavedProperties rather th

Re: RFR: 8022854 : (s) Cleaner re-initialization of System properties

2014-01-28 Thread Alan Bateman
On 27/01/2014 20:17, Mike Duigou wrote: Hello all; This is a bit of cleanup I did back during Java 8 that got deferred due to it's late arrival during the development cycle. I've updated it for Java 9. http://cr.openjdk.java.net/~mduigou/JDK-8022854/0/webrev/ This change improves the implemen

Re: RFR: 8022854 : (s) Cleaner re-initialization of System properties

2014-01-28 Thread Mandy Chung
On 1/28/14 4:06 AM, Paul Sandoz wrote: Hi Mike, Your patch is slightly out of sync with 9: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/c8c4f441fc76 This was the changeset I pushed yesterday causing this and removed "sun.lang.ClassLoader.allowArraySyntax". Sorry for the timing. htt