[Nestmates] RFR: 8188076: [Nestmates] Rename MemberOfNest attribute to NestHost

Maurizio Cimadamore maurizio.cimadamore at oracle.com
Mon Oct 16 09:49:35 UTC 2017


Hi David,
In some files (see below) some references to NestTop have been renamed - 
not sure whether this is deliberate, or just a find/replace issue

classFileParser.cpp
classFileParser.hpp
instanceKlass.cpp
instanceKlass.hpp

Can you please confirm I'm looking at the right bits? Note that your 
review email also seems inconsistent:

"The MemberOfNest attribute was renamed to NestHost. "

Ok, that matches my understanding...

"All references to nest-top now become nest-host."

That seems what's in the code, but is this correct?

Maurizio



On 13/10/17 07:19, David Holmes wrote:
> webrev: http://cr.openjdk.java.net/~dholmes/8188076/webrev/
>
> The MemberOfNest attribute was renamed to NestHost.
>
> All references to nest-top now become nest-host.
>
> There are some additional changes in 
> test/langtools/tools/javac/valhalla/nestmate/CheckNestmateAttrs.java 
> that were missed in previous commits.
>
> Thanks,
> David




More information about the valhalla-dev mailing list