RFR: 8022854 : (s) Cleaner re-initialization of System properties
Alan.Bateman at oracle.com
Tue Jan 28 15:27:13 UTC 2014
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.
> This change improves the implementation of System.setProperties(null) which restores the system properties to their boot time defaults. The existing semantics are preserved.
I agree with Paul on the naming, getSanitizedSavedProperties is a bit
odd (but not a big deal as it's internal).
On the wording then "reloaded" is better than "is forgotten" but I
wonder if it could be a bit clearer, like "reset to the default system
On the test then I remember Erik's late change to fix a build issue .
He added a simple test as part of that and maybe we should remove that
one now and maybe beef up your test to check the properties listed
System.getProperties's table are present. A minor comment is that the
date in the header suggests you wrote this test in 2013. Also just to
keep things consistent then you might want a space in "if(".
More information about the core-libs-dev