[core-libs] RFR (L): 8010319: Implementation of JEP 181: Nest-Based Access Control

[Peter Levart]

On 05/22/2018 12:36 PM, Peter Levart wrote:
> So how about putting the nestmate access logic just before the last if 
> (!successSoFar) { return false; }:
>
>         // Check for nestmate access if member is private
>         if (!successSoFar && Modifier.isPrivate(modifiers)) {
>           // assert: targetClass == null || isSubclassof(targetClass, 
> memberClass)
>           // Note: targetClass may be outside the nest, but that is okay
>           //       as long as memberClass is in the nest.
>           successSoFar = areNestMates(currentClass, memberClass);
>         }
>
>         if (!successSoFar) {
>             return false;
>         }
>
>
> This would not penalize access to package-private and protected 
> members with areNestMates() JNI calls and maybe caching will not be 
> needed at all.

Ah, just leave it as is. The if (Modifier.isPrivate(modifiers)) already 
makes sure that areNestMates is called only for private members.

Peter