On 21/02/2018 06:08, Martin Buchholz wrote:
:

8198480: Improve ClassLoaders static init block
http://cr.openjdk.java.net/~martin/webrevs/jdk/ClassLoaders-static/ <http://cr.openjdk.java.net/%7Emartin/webrevs/jdk/ClassLoaders-static/>
https://bugs.openjdk.java.net/browse/JDK-8198480

This initializer has changed a few times and I agree it's a bit awkward to understand when there is a class path or not. So I think you're rejigging is okay, as is the rewording of the comment. The blank line at L45 was intentional, no need to remove that.

-Alan

Reply via email to