RFR JDK-8066709 Make some JDK system properties read only

[Alan Bateman]

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