RFR: JDK-8161230 ClassLoader: add resource methods returning java.util.stream.Stream

Patrick Reinhart patrick at reini.net
Fri Sep 2 07:41:31 UTC 2016


On 2016-09-02 08:09, Andrej Golovnin wrote:
> src/java.base/share/classes/java/lang/ClassLoader.java
> 
> The constant RESOURCE_CHARACTERISTICS in the line 215 should be
> defined near the #streamOf()-method. The distance between the line
> 1412 where the #streamOf()-method is defined and the line 215 is just
> too huge.

No problem, moved to the method itself

> Your patch seems to modify the JavaDocs of the methods #getParent(),
> #getPlatformClassLoader(), #getSystemClassLoader(). But I don see how
> it is related to the issue you try to solve.>

Was a merge issue, removed those

> test/java/lang/ClassLoader/resources/ResourcesFailureCase.java
> test/java/lang/ClassLoader/resources/ResourcesSuccessCase.java

> The indentation is broken. You use tabs. But in JDK you must use
> spaces, see
> http://cr.openjdk.java.net/~alundblad/styleguide/index-v6.html#toc-indentation

Fixed those also.

See updated webrev:

http://cr.openjdk.java.net/~reinhapa/reviews/8161230/webrev.01

- Patrick


More information about the core-libs-dev mailing list