[GROOVY-10722] AIC via array-based (variadic) constructor null and array arguments

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 4.0.8
• Labels: varargs
• Assignee: Eric Milles
• Reporter: Eric Milles
• Created: 14/Aug/22 15:19
• Updated: 10/Feb/23 21:40
• Resolved: 18/Jan/23 18:06

Description

Follow up from GROOVY-7370. Consider the following:

class C {
  C(String... args) {
    strings = args
  }
  public String[] strings
}
def x = new C(null) { }
assert x.strings == null
def y = new String[0]
def z = new C( y ) { }
assert z.strings.length == 0

7370 handles no arguments for array and more than one. Passing null is supposed to set the array to null, not pass an array with one null element. Passing an array through an untyped variable is also a problem.

Issue Links

• is related to: GROOVY-6146 Calling a Java vararg method from Groovy with a null argument cast to the vararg type behaves differently than in Java
• is related to: GROOVY-7370 Varargs in constructor are not treated correctly when creating instances of anonymous class
• relates to: GROOVY-10840 GroovyCastException on Groovy 3 try-with-resources example after upgrade to Groovy 4