[Nestmates] RFR: 8187768 and 8189940 - nest host resolution and exceptions

[John Rose]

On Nov 2, 2017, at 6:01 AM, Maurizio Cimadamore <maurizio.cimadamore at oracle.com> wrote:
> 
> On 01/11/17 22:23, David Holmes wrote:
>> The ClassInfo has to refer to a class else it gets rejected, but it can't be a class for a primitive or array type, or ?? - it has to be an instance class. As I don't really expect anyone to ever encounter this in practice I'm not too concerned if it seems a little cryptic. The full message looks like this: 
> Right - I'm just suggesting to drop 'instance' from the message (as that seems to be inspired from instanceKlass, which doesn't have a real spec'd counterpart). Not feeling super strongly about it though.

+1 on removing 'instance'

The classfile parser has similar checks for super-class and interfaces,
so you could just adapt the error-reporting language near those checks.
For example, near line 3639 of classFileParser.cpp:

    if (need_verify) {
      guarantee_property(!is_array,
                        "Bad superclass name in class file %s", CHECK_NULL);
    }


— John