RFR: 8193037 package-info annotations are not reported when annotation processing is enabled

Liam Miller-Cushon cushon at google.com
Thu Sep 13 22:38:15 UTC 2018


Hello,

Please consider this fix for JDK-8193037. Currently
JavacProcessingEnvironment incorrectly resets package-infos corresponding
to all package in the compilation (even if the package-info was loaded from
the classpath), rather than only resetting package-infos that are being
compiled from source as part of that compilation.

The fix uses the isPkgInfo helper already defined in
JavacProcessingEnvironment.

webrev: http://cr.openjdk.java.net/~cushon/8193037/webrev.01/
bug: https://bugs.openjdk.java.net/browse/JDK-8193037

On Thu, Jan 25, 2018 at 10:27 AM Liam Miller-Cushon <cushon at google.com>
wrote:

> Friendly ping.
>
> On Mon, Jan 8, 2018 at 11:42 AM, Liam Miller-Cushon <cushon at google.com>
> wrote:
>
>> Is anyone interested in sponsoring this patch?
>>
>> On Tue, Dec 5, 2017 at 5:49 PM, Liam Miller-Cushon <cushon at google.com>
>> wrote:
>>
>>> Hello,
>>>
>>> Please review the following fix for JDK-8193037.
>>>
>>> javac should only reset package-info symbols between annotation
>>> processing rounds if
>>> the package-info is being compiled from source. Previously package-infos
>>> loaded from
>>> the classpath were reset and not reinitialized, leading to package
>>> annotations being
>>> unavailable during later annotation processing rounds.
>>>
>>> I included an additional test case to ensure that package-infos are
>>> still reset when
>>> necessary. Related discussion:
>>>
>>> http://mail.openjdk.java.net/pipermail/compiler-dev/2017-December/011398.html
>>>
>>> bug: https://bugs.openjdk.java.net/browse/JDK-8193037
>>> webrev: http://cr.openjdk.java.net/~cushon/8193037/webrev.00/
>>>
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/compiler-dev/attachments/20180913/725e8a46/attachment-0001.html>


More information about the compiler-dev mailing list