RFR: 8254023: A module declaration is not allowed to be a target of an annotation that lacks an @Target meta-annotation
Vicente Romero
vromero at openjdk.java.net
Mon Dec 7 15:55:19 UTC 2020
On Thu, 3 Dec 2020 08:20:13 GMT, Guoxiong Li <github.com+13688759+lgxbslgx at openjdk.org> wrote:
>> Thanks for focusing on this patch. I will continue to follow the discussion and revise my patch based on the outcome of the discussion.
>
>> Short term (Java 16) it makes sense to fix https://bugs.openjdk.java.net/browse/JDK-8254023 given that both competing specs agree for this case, it is a small fix, and externally found.
>>
>>For Java 17 we should align both specs and the reference implementation one way or another.
>
> @jbf According to your comments, I revise my code to only fix JDK-8254023.
> The rest of the work, ambiguity of the specifications, will be clarified and fixed in Java 17.
> Thank you for taking the time to review.
the change looks good but I'm surprised that no other tests are failing and need changes, in particular tests like: `test/langtools/tools/javac/annotations/repeatingAnnotations/combo/TargetAnnoCombo.java`
-------------
PR: https://git.openjdk.java.net/jdk/pull/622
More information about the compiler-dev
mailing list