RFR: addressing several issues in the implementation of universal tvars [v6]
Vicente Romero
vromero at openjdk.java.net
Tue Jun 7 18:40:04 UTC 2022
On Tue, 7 Jun 2022 14:37:51 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
>> Vicente Romero has updated the pull request incrementally with one additional commit since the last revision:
>>
>> adding structural type comparator to detect unchecked conversions
>
> src/jdk.compiler/share/classes/com/sun/tools/javac/code/Types.java line 1179:
>
>> 1177: case BOT: case NONE:
>> 1178: return t.hasTag(s.getTag());
>> 1179: case WILDCARD: //we shouldn't be here - avoids crash (see 7034495)
>
> I do not see a visitor method for wildcards, but I think you need one, as you might get to wildcards recursively, because of `visitClassType`
I think we should be covered here as visitors: `StructuralTypeComparator` and `TypeArgStructComparator` are interdependent, so I think we should be fine there, unless I'm missing something
-------------
PR: https://git.openjdk.java.net/valhalla/pull/684
More information about the valhalla-dev
mailing list