RFR JDK-8066709 Make some JDK system properties read only
Roger Riggs
Roger.Riggs at Oracle.com
Mon Jun 18 14:31:45 UTC 2018
Hi Alan,
Thanks for the review and suggestion.
The CSR and Webrev are updated.
webrev:
http://cr.openjdk.java.net/~rriggs/webrev-static-property-8066709/
csr:
https://bugs.openjdk.java.net/browse/JDK-8204235
On 6/16/2018 8:42 AM, Alan Bateman wrote:
> On 13/06/2018 15:10, Roger Riggs wrote:
>> Hi Joe,
>>
>> The CSR text is still in draft and got out of sync with the open
>> review and comments.
>>
>> The updated apiNote clarifies that changes made through the
>> System.*property* methods
>> may not affect the value cached during initialization.
>> The implementation changes affect a very short list of properties.
>>
>> The note on the methods is to raise awareness that individual
>> properties may have
>> different behavior and may be unpredictable.
> Just catching up on this. I checked the CSR and the webrev (the latest
> webrev was generated on June 6 so I hope that is the version we are
> meant to look at).
>
> The updated apiNote (CSR version) looks okay but I think the word
> "internal" needs to be dropped as it comes with too many questions.
> "may be cached during initialization or ..." should be okay. The
> editing has meant the line lengths are a bit inconsistent so I assume
> they can be fixed up before the change is pushed.
>
> I see the original patch to SocksSocketImpl has been mostly reverted.
> It looks correct now. The other usages look okay to me.
>
> -Alan
More information about the core-libs-dev
mailing list