RFR: More cleanup patches

[Martin Buchholz]

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

> 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
saying?!