Hi David,

I'm reviewing the core reflection and j.l.invoke spec change.

I think Class::getNestHost and Class::getNestMembers should do a security
permission check as performed in other method e.g. getEnclosingClass.

     * @throws SecurityException
     *         If a security manager, <i>s</i>, is present and
     *         the caller's class loader is not the same as or an
     *         ancestor of the class loader for the current class and
     *         invocation of {@link SecurityManager#checkPackageAccess
     *         s.checkPackageAccess()} denies access to the package
     *         of this class
     *

I suggest to make @apiNote in Class::getNestHost as just part of javadoc.
It can define a section like "Nest Membership" that can be referenced
by Lookup class spec.  I also have some suggested edits that you can
consider:

http://cr.openjdk.java.net/~mchung/valhalla/webrevs/nestmates-spec/

Mandy

On 3/6/18 10:12 PM, David Holmes wrote:
The Nestmates CSR request:

https://bugs.openjdk.java.net/browse/JDK-8197445

has been prepared by Dan Smith and myself. Before Proposing this CSR request it needs to have Reviewers add themselves to it.

(The first reviewer will need to edit the issue to enter their OpenJDK user name in the "Reviewed by" field. Subsequent reviewers can simply click on the "Reviewed by" field in the "People" section and add their user name.

The CSR contains links to all the updated specification documents, all of which have been previously sent out for review/comment to the EG (and observers).

Thanks,
David

Reply via email to