groovy

Calling varargs method from Groovy creates varargs array of different type compared to Java

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.6.3
  • Fix Version/s: 1.9-beta-4, 1.8.4, 1.7.11
  • Component/s: Compiler
  • Labels:
    None
  • Testcase included:
    yes
  • Number of attachments :
    0

Description

Whereas Java creates an array whose runtime type matches the vararg's static type, Groovy creates an array whose runtime type matches the argument's type. Although it's not clear (to me) if the JLS is prescriptive in this area, Groovy should probably do the same as Java.

All of the following tests currently fail:

class VarArgsTest extends GroovyTestCase {
  void testCallObjectVarArgWithInt() {
    assert foo(1).getClass() == Object[] // actual: Integer[]
  }

  void testCallObjectVarArgWithStrings() {
    assert foo("one", "two").getClass() == Object[] // actual: String[]
  }

  void testCallObjectVarArgImplementedInJavaWithInt() {
    assert JavaVarArgsMethod.foo(1).getClass() == Object[] // actual: Integer[]
  }

  void testCallObjectVarArgImplementedInJavaWithStrings() {
    assert JavaVarArgsMethod.foo("one", "two").getClass() == Object[] // actual: String[]
  }

  void testCallSerializableVarArgWithString() {
    assert bar("").getClass() == Serializable[] // actual: String[]
  }

  def foo(Object... args) {
    args
  }

  def bar(Serializable... args) {
    args
  }
}

JavaVarArgsMethod.java:

public class JavaVarArgsMethod {
  public static Object foo(Object... args) {
    return args;
  }
}

Activity

blackdrag blackdrag made changes -
Field Original Value New Value
Resolution Fixed [ 1 ]
Fix Version/s 1.9-beta-4 [ 17656 ]
Fix Version/s 1.8.4 [ 17852 ]
Fix Version/s 1.7.11 [ 17244 ]
Assignee Jochen Theodorou [ blackdrag ]
Status Open [ 1 ] Closed [ 6 ]

People

  • Assignee:
    blackdrag blackdrag
    Reporter:
    Peter Niederwieser
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: