RFR: 8273408: java.lang.AssertionError: typeSig ERROR on generated class property of record [v2]

Guoxiong Li gli at openjdk.java.net
Wed Sep 15 06:16:43 UTC 2021


On Wed, 15 Sep 2021 01:43:17 GMT, Vicente Romero <vromero at openjdk.org> wrote:

>> Guoxiong Li has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Treat source as comment mistakenly. Remove the comment symbol.
>
> src/jdk.compiler/share/classes/com/sun/tools/javac/code/Symbol.java line 1517:
> 
>> 1515:                         // If the class type of the record component is generated by annotation processor, it should
>> 1516:                         // use the new actual class type and symbol instead of the old dummy ErrorType.
>> 1517:                         // toRemove = rc;
> 
> there is dead code here, `toRemove` doesn't seem to be assigned ever, probably you uploaded an intermediate patch?

The `// toRemove = rc;` should be `toRemove = rc;`. It is a statement instead of a comment. I revised it mitakenly in my second commit `Polish`. In the second commit `Polish`, I only want to fix the indentation so that I didn't test again after revising. Unfortunately, the IDE changed `toRemove = rc;` to a comment. Apology for the mistake.

-------------

PR: https://git.openjdk.java.net/jdk/pull/5511


More information about the compiler-dev mailing list