Direct method calls with vargs throws AIOBE

DirectMethodCallWithVargsTest.groovy

package groovy.bugs

import org.codehaus.groovy.ast.ClassCodeVisitorSupport
import org.codehaus.groovy.control.SourceUnit
import org.codehaus.groovy.ast.expr.MethodCallExpression
import org.codehaus.groovy.ast.MethodNode
import org.codehaus.groovy.control.CompilerConfiguration
import org.codehaus.groovy.control.customizers.CompilationCustomizer
import org.codehaus.groovy.control.CompilePhase
import org.codehaus.groovy.classgen.GeneratorContext
import org.codehaus.groovy.ast.ClassNode

class DirectMethodCallWithVargsTest extends GroovyTestCase {
    
    void testDirectMethodCallWithVargs() {
        def config = new CompilerConfiguration()
        config.addCompilationCustomizers(
                new MyCustomizer()
        )
        GroovyShell shell = new GroovyShell(config)
        shell.evaluate '''
            def foo(String... args) {
                (args as List).join(',')
            }
            assert foo('1','2','3') == '1,2,3'
        '''
    }

    private static class MyCustomizer extends CompilationCustomizer {

        MyCustomizer() {
            super(CompilePhase.CANONICALIZATION)
        }

        @Override
        void call(final SourceUnit source, final GeneratorContext context, final ClassNode classNode) {
            def visitor = new MethodCallVisitor(source)
            classNode.methods.each { visitor.visitMethod(it) }
            visitor.visitClass(classNode)
        }
    }
    
    private static class MethodCallVisitor extends ClassCodeVisitorSupport {
        private final SourceUnit unit
        private MethodNode fooMethod
        
        MethodCallVisitor(SourceUnit source) {
            unit = source
        }
        
        @Override
        protected SourceUnit getSourceUnit() {
            return unit
        }

        @Override
        void visitMethod(final MethodNode node) {
            super.visitMethod(node)
            if (node.name=='foo') {
                fooMethod = node
            }
        }


        @Override
        void visitMethodCallExpression(final MethodCallExpression call) {
            super.visitMethodCallExpression(call)
            if (call.methodAsString=='foo') {
                call.methodTarget = fooMethod
            }
        }


    }
}