[12] RFR: 8214567 and 8214569: Use {@systemProperty} for definitions of system properties

[Deepak Kejriwal]

Hi Mandy / Alan,

 

Thanks for review. I have modified the ClassLoader.java as per the given comments. 

 

Please find updated version of webrev:-

 

http://cr.openjdk.java.net/~rpatil/8214567%2b8214569/webrev.01/

 

Regards,

Deepak

 

From: Mandy Chung 
Sent: Friday, December 21, 2018 10:05 PM
To: Deepak Kejriwal <deepak.kejriwal at oracle.com>
Cc: core-libs-dev <core-libs-dev at openjdk.java.net>
Subject: Re: [12] RFR: 8214567 and 8214569: Use {@systemProperty} for definitions of system properties

 

 

On 12/21/18 1:49 AM, Deepak Kejriwal wrote:

Hi all,
 
Please review the fix for following issues:-
 
https://bugs.openjdk.java.net/browse/JDK-8214567
 
https://bugs.openjdk.java.net/browse/JDK-8214569
 
Below is the webrev for above issues:
 
http://cr.openjdk.java.net/~rpatil/8214567%2b8214569/webrev.00/
 


Looks okay.

Like Alan suggests, reformating line 1867 in ClassLoader.java  to avoid that single long line sticking out would be good.

Mandy