[12] RFR: 8214567 and 8214569: Use {@systemProperty} for definitions of system properties
Deepak Kejriwal
deepak.kejriwal at oracle.com
Mon Dec 24 09:17:30 UTC 2018
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
More information about the core-libs-dev
mailing list