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

Vicente Romero vromero at openjdk.org
Thu Oct 24 19:52:11 UTC 2024


On Sat, 19 Oct 2024 11:57:08 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 with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 20 additional commits since the last revision:
> 
>  - Merge branch 'openjdk:master' into selaed-permits-allow-private-members
>  - remove logging error
>  - Merge remote-tracking branch 'origin/selaed-permits-allow-private-members' into selaed-permits-allow-private-members
>  - Revert bugid change
>  - Change bugid
>  - report feature error
>  - add feature
>  - make method names shorter
>  - fix test failures 3
>  - fix test failures 2
>  - ... and 10 more: https://git.openjdk.org/jdk/compare/e8b21c9c...38dd1b5a

minor suggestion, other than that, looks good, thanks for fixing this

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

Marked as reviewed by vromero (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/20718#pullrequestreview-2393577301


More information about the compiler-dev mailing list