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

Eric Milles commented on GROOVY-9244:
-------------------------------------

Here is the bytecode for the anon. inner constructor in case that helps anyone 
understand what might be changed:
{code}
  // Method descriptor #13 (LScript;Ljava/lang/Object;)V
  // Stack: 5, Locals: 4
  synthetic Script$1(Script p0, java.lang.Object p10);
      0  aload_1 [p0]
      1  astore_3
      2  aload_3
      3  aload_0 [this]
      4  swap
      5  putfield Script$1.this$0 : Script [17]
      8  aload_3
      9  pop
     10  iconst_1
     11  anewarray java.lang.Object [19]
     14  dup
     15  iconst_0
     16  aload_2 [p10]
     17  aastore
     18  dup
     19  bipush -1
     21  ldc <Class Groovy3> [4]
     23  invokestatic 
org.codehaus.groovy.runtime.ScriptBytecodeAdapter.selectConstructorAndTransformArguments(java.lang.Object[],
 int, java.lang.Class) : int [25]
{code}

> Anonymous subclasses should cast their super-parameters
> -------------------------------------------------------
>
>                 Key: GROOVY-9244
>                 URL: https://issues.apache.org/jira/browse/GROOVY-9244
>             Project: Groovy
>          Issue Type: Bug
>    Affects Versions: 2.4.7, 2.5.7
>         Environment: I tried in GroovyConsole 2.4.7 and Groovy web console 
> 2.5.7
>            Reporter: Björn Kautler
>            Priority: Major
>
> Given this code:
> {noformat}
>     import java.util.regex.Pattern
>     abstract class Groovy3 {
>         Groovy3(String s) { println "String" }
>         Groovy3(Pattern p) { println "Pattern" }
>     }
>     class Groovy4 extends Groovy3 {
>         Groovy4(String s) { super(s) }
>         Groovy4(Pattern p) { super(p) }
>     }
>     class Groovy5 extends Groovy3 {
>         Groovy5(String s) { super(s as String) }
>         Groovy5(Pattern p) { super(p as Pattern) }
>     }
>     class Groovy6 {
>         Groovy6(String s) { println "String" }
>         Groovy6(Pattern p) { println "Pattern" }
>     }
>     new Groovy3(null as String) { }
>     new Groovy4(null as String)
>     new Groovy5(null as String)
>     new Groovy6(null as String)
> {noformat}
> Groovy3ish and Groovy4 instantiation fail
> Groovy5 and Groovy6 instantiation succeed and print {{String}}.
> I did not find a way to make the Groovy3ish one work, except for doing it the 
> Groovy4 way, but no chance with an anonymous subclass.
> From what I observed, I guess the anonymous subclass does it like Groovy4 but 
> should do it like Groovy5 to properly select the super constructor.



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

Reply via email to