RFR: More cleanup patches

Martin Buchholz martinrb at google.com
Fri Mar 30 22:07:21 UTC 2018

Maybe I'm starting to understand.

            // attempt to load class in module defined to this loader
            assert loadedModule.loader() == this;
            return findClassInModuleOrNull(loadedModule, cn);

Precondition of calling findClassInModuleOrNull is that
loadedModule.loader() == this;
so perhaps move that assert into the body of findClassInModuleOrNull ?


        // no class loading until VM is fully initialized
        if (!VM.isModuleSystemInited())

Here the method call is so self-documenting that we should just delete the
"VM is fully initialized" is not the same as "module system is

More information about the core-libs-dev mailing list