[jira] [Assigned] (GROOVY-9245) synthetic constructors should be ignored

["Eric Milles (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/GROOVY-9245?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Eric Milles reassigned GROOVY-9245:
-----------------------------------

    Assignee: Eric Milles

> synthetic constructors should be ignored
> ----------------------------------------
>
>                 Key: GROOVY-9245
>                 URL: https://issues.apache.org/jira/browse/GROOVY-9245
>             Project: Groovy
>          Issue Type: Bug
>            Reporter: Björn Kautler
>            Assignee: Eric Milles
>            Priority: Major
>
> Groovy should ignore synthetic constructors when instantiating objects, or at most fall back to them if no non-synthetic one is found.
> If you have a Java class with a private constructor and inner classes (doesn't matter whether static or non-static) that are subclasses of the outer class and call one of the private super-constructors (for example by not calling {{super}} explicitly and having a parameterless private constructor in the super class, or by calling any private constructor explicitly using {{super}}), an additional synthetic package-private constructor with a synthetic anonymous inner class of the outer class as additional argument that then forwards the call to the private constructor is created by the compiler so that the inner class has something non-private to call as super constructor.
> So assuming you have this Java class:
> {code:java}
> class Foo {
>     private Foo() {
>     }
>     public Foo(String foo) {
>     }
>     private static class Bar extends Foo {
>     }
> }
> {code}
> and then from Groovy try to call `new Foo(null)` you get the quite unexpected (unless you are aware of compiler internals)
> {noformat}
> Ambiguous method overloading for method net.kautler.command.api.restriction.Foo#<init>.
> Cannot resolve which method to invoke for [null] due to overlapping prototypes between:
> 	[class net.kautler.command.api.restriction.Foo$1]
> 	[class java.lang.String]
> {noformat}
> Or with this:
> {code:java}
> class Foo {
>     private Foo() {
>     }
>     private Foo(String foo) {
>     }
>     public Foo(String foo, String bar) {
>     }
>     private static class Bar extends Foo {
>         public Bar() {
>             super(null);
>         }
>     }
> }
> {code}
> and you try to do from Groovy `new Foo(null, null)`, you get
> {noformat}
> Ambiguous method overloading for method net.kautler.command.api.restriction.Foo#<init>.
> Cannot resolve which method to invoke for [null, null] due to overlapping prototypes between:
> 	[class java.lang.String, class net.kautler.command.api.restriction.Foo$1]
> 	[class java.lang.String, class java.lang.String]
> {noformat}
> I'd suggest when searching for a constructor to invoke to first filter out the synthetic constructors (or at least the synthetic constructors with a synthetic anonymous inner class as last paramter) and only if none was found, search in the synthetic ones.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)