Re: Java/12725: Binary Compatibility

[Tom Tromey <tromey at redhat dot com>]

>>>>> "Andrew" == Andrew Haley <aph@redhat.com> writes:

>> I think both these places should pass `klass->getClassLoaderInternal()'
>> as the second argument instead.

Andrew> That would be my guess.  I need a more sophisticated test suite to be
Andrew> certain.  I suppose we could just make the change anyway.

Yeah.  We don't have tests for most of the class loader changes,
aside from running applications using the code.

The general rule, though, is "a class loaded on behalf of a class K
must be loaded using K's class loader".  E.g., stuff in the constant
pool (including stuff like instanceof, new, etc), Class.forName()
without the loader argument, symbolic `.class' references, etc, fall
into this rule.

Tom