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

[David Holmes]

Thanks Bernard!

David

On 8/06/2018 2:36 AM, B. Blaser wrote:
> Hi David,
> 
> On 7 June 2018 at 12:02, David Holmes <david.holmes at oracle.com> wrote:
>> Hi compiler-team,
>>
>> Was just getting ready to push this when I discovered a newly added test
>> doesn't pass under nestmates:
>>
>> tools/javac/classfiles/attributes/Synthetic/AccessToPrivateInnerClassConstructorsTest.java
>>
>> webrev:
>> http://cr.openjdk.java.net/~dholmes/8010319-JEP181/webrev.langtools.v5-incr/
>>
>> Fix was to reduce expected number of synthetic methods from 2 to 0, and
>> expected number of synthetic classes from 1 to 0. This is similar to a
>> previous change:
>>
>> http://cr.openjdk.java.net/~dholmes/8010319-JEP181/webrev.langtools.v1/test/langtools/tools/javac/classfiles/attributes/Synthetic/AccessToPrivateInnerClassMembersTest.java.cdiff.html
>>
>> Thanks,
>> David
> 
> Looks good to me too.
> I recently added this test while correcting an ancient bug which will
> be implicitly solved by JEP 181 if target >= 11.
> 
> Cheers,
> Bernard
>