RFR: 8026369: javac potentially ambiguous overload warning needs an improved scheme [v4]
Archie L. Cobbs
duke at openjdk.org
Sat Feb 25 03:57:31 UTC 2023
On Fri, 24 Feb 2023 22:24:10 GMT, Vicente Romero <vromero at openjdk.org> wrote:
>> Archie L. Cobbs has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Fix typo in one comment and clarify another.
>
> src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Check.java line 2730:
>
>> 2728: types.membersClosure(site, false).getSymbols(new ClashFilter(site), RECURSIVE).spliterator(), false)
>> 2729: .map(MethodSymbol.class::cast)
>> 2730: .filter(m -> m.owner.type.tsym != syms.objectType.tsym)
>
> nit: this filter could be folded with the one above, `new ClashFilter(site)`
Should be fixed in 7d8a54c6679.
> src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Check.java line 2772:
>
>> 2770: java.util.List<MethodSymbol> overriddenMethods = list.stream()
>> 2771: .filter(m2 -> m2 != m)
>> 2772: .filter(m2 -> overrides.test(m, m2))
>
> can these two filters be folded?
Should be fixed in 7d8a54c6679.
-------------
PR: https://git.openjdk.org/jdk/pull/12645
More information about the compiler-dev
mailing list