There should be a way of generating a Java-style varargs method signature from a Scala sequence method

[scabug]

If I declare a method in a class in "scala-sequence" style:

    def foo(args: String*) : Unit

And inspect the generated class file (as if it had been generated from a Java class), the signature looks like:

    public void foo(scala.collection.Seq<java.lang.String> p0)

I think that, for interop with Java, it would be nice if we could generate a Java varargs method signature (that is, taking a String array with the transient modifier set):

    public transient void foo(java.lang.String[] p0)

I tried adding the @transient annotation but this was ignored by scalac!

I would ask for either:

1. The compiler automatically generates Java varargs-style "forwarder" methods for Scala sequence ones

2. A @VarArgs annotation is supplied, which, when used in conjunction with a scala-sequence method, generates a Java varargs forwarder as in Scaladoc NullPointerException #1

Chris

[scabug]

Imported From: https://issues.scala-lang.org/browse/SI-3621?orig=1
Reporter: @oxbowlakes

[scabug]

@oxbowlakes said:
Oops - sorry for the bad link to the first bullet point

[scabug]

@axel22 said:
Working on it.

One thing troubles me, though. Why do you want to mark the method as transient? Looking through the Java language and vm specifications, and browsing all around the net, I couldn't find a reference for the transient modifier being applied to methods and what that means.

[scabug]

@oxbowlakes said:
As far as I was aware, Java used the transient modifier within the bytecode to mark a method which takes an array as being a varargs method.

I now cannot find any reference to this, so I am presumably wrong. However, IDEA will mark a varargs method in a class file as being "transient", so perhaps that's the source of the issue.

[scabug]

@oxbowlakes said:
I take it all back - this is easily testable as follows:

import java.lang.reflect.Method;
import java.lang.reflect.Modifier;

public class VarargsTest {
public static void main(String[] args) {
for (Method method : VarargsTest.class.getDeclaredMethods()) {
System.out.println(method.getName() + " " + ((method.getModifiers() & Modifier.TRANSIENT) > 0 ? "TRANSIENT" : ""));
}

}

public static void foo(String[] ss) {
    System.out.println("ss = " + ss);
}

public static void bar(String... s) {
    System.out.println("s = " + s);
}

}

[scabug]

@axel22 said:
Interesting. In fact, I was wrong - it seems to be documented here:

http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6516895

Ok, then so be it.

[scabug]

@dcsobral said:
Wasn't this closed by r23578?

[scabug]

@axel22 said:
I'm still waiting for the review.

[scabug]

@paulp said:
I reckon that's all the reviewin' we're gettin'.