RFR: 8273408: java.lang.AssertionError: typeSig ERROR on generated class property of record [v3]
Vicente Romero
vromero at openjdk.java.net
Thu Sep 16 15:15:46 UTC 2021
On Thu, 16 Sep 2021 12:24:28 GMT, Guoxiong Li <gli at openjdk.org> wrote:
>> Hi all,
>>
>> After processing annotation, if new source files are generated, the compiler should re-parse and re-enter the source files so that the type and symbol can keep right. The compiler always holds this convention in the past. But when the feature `record` is added, the convention is broken. In this bug, the compiler doesn't catch or enter the new/good type for record component after processing annotation. So some issues may occur at the later phases.
>>
>> This patch fixes the type of the record component so that the phases `Gen` and `ClassWriter` can get the right type. And a test case is added.
>>
>> Thanks for taking the time to review.
>>
>> Best Regards,
>> -- Guoxiong
>
> Guoxiong Li 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 six additional commits since the last revision:
>
> - Add test cases about unknown type and annotation.
> - Fix comments.
> - Merge branch 'master' into JDK-8273408
> - Treat source as comment mistakenly. Remove the comment symbol.
> - Polish
> - 8273408: java.lang.AssertionError: typeSig ERROR on generated class property of record
looks good, thanks for the additional research
-------------
Marked as reviewed by vromero (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/5511
More information about the compiler-dev
mailing list