RFR: JDK-8315458 Implement JEP 463: Implicitly Declared Classes and Instance Main Method (Second Preview) [v42]
Jan Lahoda
jlahoda at openjdk.org
Wed Nov 29 14:51:33 UTC 2023
On Wed, 29 Nov 2023 14:30:50 GMT, Jim Laskey <jlaskey at openjdk.org> wrote:
>> Address changes from JEP 445 to JEP 463.
>>
>> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class.
>>
>> - Don't mark class on read.
>>
>> - Remove reflection and annotation processing related to unnamed classes.
>>
>> - Simplify main method search.
>
> Jim Laskey has updated the pull request incrementally with one additional commit since the last revision:
>
> Windows line endings
javac changes seem reasonably to me. Adding two nits to `InstanceMainTest.java`.
test/jdk/tools/launcher/InstanceMainTest.java line 45:
> 43: """,
> 44:
> 45: // instance dominating instance
Nit:
Suggestion:
// instance dominating static
test/jdk/tools/launcher/InstanceMainTest.java line 84:
> 82: """,
> 83:
> 84: // unnamed class dominating instance
Nit:
Suggestion:
// main with args dominating main without args
-------------
Marked as reviewed by jlahoda (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/16461#pullrequestreview-1755452564
PR Review Comment: https://git.openjdk.org/jdk/pull/16461#discussion_r1409394109
PR Review Comment: https://git.openjdk.org/jdk/pull/16461#discussion_r1409395226
More information about the compiler-dev
mailing list