groovy-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (GROOVY-7922) Static type checking not strict enough in the presence of ambiguous method matching
Date Sun, 18 Sep 2016 06:53:22 GMT

    [ https://issues.apache.org/jira/browse/GROOVY-7922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15500423#comment-15500423
] 

ASF GitHub Bot commented on GROOVY-7922:
----------------------------------------

Github user blindpirate commented on a diff in the pull request:

    https://github.com/apache/groovy/pull/422#discussion_r79298948
  
    --- Diff: src/main/org/codehaus/groovy/transform/stc/StaticTypeCheckingSupport.java ---
    @@ -1157,6 +1142,45 @@ private static ClassNode makeRawType(final ClassNode receiver)
{
             return result;
         }
     
    +    private static void removeMethodInSuperInterface(List<MethodNode> toBeRemoved,
MethodNode one, MethodNode two) {
    +        ClassNode oneDC=one.getDeclaringClass();
    +        ClassNode twoDC=two.getDeclaringClass();
    +        if(oneDC.implementsInterface(twoDC)){
    +            toBeRemoved.add(two);
    +        }else{
    +            toBeRemoved.add(one);
    +        }
    +    }
    +
    +    private static boolean areEquivalentInterfaceMethods(MethodNode one, MethodNode two,
Parameter[] onePars, Parameter[] twoPars) {
    --- End diff --
    
    Yes, good suggestion. I will do that.


> Static type checking not strict enough in the presence of ambiguous method matching
> -----------------------------------------------------------------------------------
>
>                 Key: GROOVY-7922
>                 URL: https://issues.apache.org/jira/browse/GROOVY-7922
>             Project: Groovy
>          Issue Type: Bug
>          Components: Static Type Checker
>            Reporter: Paul King
>
> This example:
> {code}
> import groovy.transform.CompileStatic
> interface FooA {}
> interface FooB {}
> class FooAB implements FooA, FooB {}
> @CompileStatic
> class TestGroovy {
>     static void test() { println new TestGroovy().foo(new FooAB()) }
>     def foo(FooB x) { 43 }
>     def foo(FooA x) { 42 }
> }
> TestGroovy.test()
> {code}
> Should probably throw some kind of ambiguous method error during compilation to match
Java (and dynamic Groovy's runtime error).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message