RFR: 8230827 javac gives inappropriate warning about potentially ambiguous methods
Adam Sotona
adam.sotona at oracle.com
Thu May 28 08:58:38 UTC 2020
Thanks for the review.
May I ask you for integration of the patch :) here is changeset version of the webrev:
http://cr.openjdk.java.net/~asotona/8230827/webrev.01/
Thanks,
Adam
> On 28 May 2020, at 10:45, Jan Lahoda <jan.lahoda at oracle.com> wrote:
>
> Seems reasonable to me.
>
> Jan
>
> On 26. 05. 20 8:08, Adam Sotona wrote:
>> Hi,
>> I would like to ask for review of the patch fixing com.sun.tools.javac.comp.Comp:checkPotentiallyAmbiguousOverloads. The actual implementation is too strict and is warning about ambiguous methods even they differ in following parameters.
>> Plus I've created a test for ambiguous and non-ambiguous cases.
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8230827
>> webrev: http://cr.openjdk.java.net/~asotona/8230827/webrev.00/
>> Mach 5 build with the patch passes all Tier 1 and Tier 2 tests.
>> Several unrelated Tier 3 tests are actually failing.
>> Thanks for the review,
>> Adam
More information about the compiler-dev
mailing list