RFR: 8338981: Access to private classes should be permitted inside the permits clause of the enclosing top-level class [v18]

Vicente Romero vromero at openjdk.org
Mon Oct 28 18:18:15 UTC 2024


On Sat, 26 Oct 2024 14:04:55 GMT, Evemose <duke at openjdk.org> wrote:

>> Fix involves adding new flag to after context that indicates that currently resolved symbol is in permits clause
>
> Evemose has updated the pull request incrementally with three additional commits since the last revision:
> 
>  - remove trailing whitespaces
>  - Merge remote-tracking branch 'origin/selaed-permits-allow-private-members' into selaed-permits-allow-private-members
>  - address review comments

test/langtools/tools/javac/sealed/PrivateMembersInPermitClause.java line 80:

> 78:      */
> 79:     @Test
> 80:     public void otherTopLevelPrivateClassFails() throws Exception {

this test is failing for me (Linux):


Expected errors: [S.java:5:25: compiler.err.report.access: S.A, private, S, 1 error], 
but got: [S.java:4:25: compiler.err.report.access: S.A, private, S, 1 error]

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/20718#discussion_r1819529873


More information about the compiler-dev mailing list