RFR: 8261088: Repeatable annotations without @Target cannot have containers that target module declarations

[Liam Miller-Cushon]

Any takers?

I think this is a relatively low-risk (and admittedly low-priority) change,
but it also has some code health value in that it deduplicates logic
related to @Target-less annotations.

There are currently two different lists of default @Target elements (one of
which is used for handling repeatable annotations, one for non-repeatable
annotations), and the lists are out of sync. Fixing this may reduce the
risk that they get further out of sync as new element kinds are added.

On Wed, Apr 21, 2021 at 10:31 AM Liam Miller-Cushon <cushon at google.com>
wrote:

> Hello, is there any additional feedback on this change or the associated
> CSR (JDK-8261181 <https://bugs.openjdk.java.net/browse/JDK-8261181>)?
>
> On Thu, Feb 4, 2021 at 12:59 PM Liam Miller-Cushon <
> cushon at openjdk.java.net> wrote:
>
>> Please review this fix to consolidate handling of default `@Target`s for
>> repeated annotations, and permit repeatable annotations without an
>> `@Target` to have container annotations that explicitly target `MODULE`.
>>
>> -------------
>>
>> Commit messages:
>>  - 8261088: Repeatable annotations without @Target cannot have containers
>> that target module declarations
>>
>> Changes: https://git.openjdk.java.net/jdk/pull/2412/files
>>  Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2412&range=00
>>   Issue: https://bugs.openjdk.java.net/browse/JDK-8261088
>>   Stats: 88 lines in 4 files changed: 66 ins; 16 del; 6 mod
>>   Patch: https://git.openjdk.java.net/jdk/pull/2412.diff
>>   Fetch: git fetch https://git.openjdk.java.net/jdk
>> pull/2412/head:pull/2412
>>
>> PR: https://git.openjdk.java.net/jdk/pull/2412
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/compiler-dev/attachments/20210510/803cf240/attachment.htm>