RFR: More cleanup patches

Martin Buchholz martinrb at google.com
Fri Mar 30 21:23:38 UTC 2018

On Fri, Mar 30, 2018 at 3:53 AM, Alan Bateman <Alan.Bateman at oracle.com>

> On 30/03/2018 02:55, Martin Buchholz wrote:
>> 8200125: Fix some classloader/module typos
>> http://cr.openjdk.java.net/~martin/webrevs/jdk/loader-typos/
>> https://bugs.openjdk.java.net/browse/JDK-8200125
> Most of these are okay but you've dropped the text "defined to this
> ClassLoader" from the findClassInModuleOrNull methods. Can we leave that
> text as they really mean "this" class loader, these methods do not delegate.

 Hmmm... I admit to being fuzzy on what's going on here, but:
findClassInModuleOrNull(loadedModule, className)
is essentially the same as
defineClass(className, loadedModule)
(why are the arguments reversed?)
so their docs should be aligned.  Both are loading from the given
LoadedModule, so it seems "defined to this ClassLoader" goes without

More information about the core-libs-dev mailing list