RFR JDK-8066709 Make some JDK system properties read only
Roger Riggs
roger.riggs at oracle.com
Wed Jun 27 02:10:39 UTC 2018
Hi,
Updated webrev:
http://cr.openjdk.java.net/~rriggs/webrev-static-property-8066709/index.html
Applied changes from prior comments and droped a change no longer needed
due
to the TLS 1.3 removal of ClientKeyExchangeService.java.
The CSR has been approved without possibly confusing @implNote in
System.getProperties
about caching of specific properties, including java.home, etc.
Thanks for any additional comments.
Roger
On 6/19/18 11:52 AM, Brent Christian wrote:
> On 6/19/18 8:08 AM, Roger Riggs wrote:
>>>
>>> * src/java.base/share/classes/java/lang/System.java :
>>>
>>> Should the @implNote with the list of cached properties be added
>>> everywhere the @apiNote is being added ? Right now the @implNote is
>>> only added to getProperties().
>>>
>> The repetition was getting tiresome and the base of all the
>> xxxProperties methods is getProperties.
>> Joe suggested having one copy of the full information and
>> referring to that from the individual @apiNotes.
>
> Fair enough.
>
>>> * src/java.base/share/classes/jdk/internal/util/StaticProperty.java :
>>>
>>> 45 private StaticProperty() {
>>> 46
>>> 47 }
>>>
>>> Maybe put this all on one line?
>>>
>> Will do
>
> Thanks,
> -Brent
>
More information about the core-libs-dev
mailing list