Janino throws runtime excetion for a method with more than 254 of operand stack height

[kiszk]

This is the same as issue #6 in the old repository

When we pass the following code piece to Janino, Janino throws the following exception because the max height of Java operand stack for foo() is more than 254.
This issue was revealed by the JIRA entry

Program

double test(double d0, double d1, ..., double d126) { return 1.0D; }

double foo() {
  boolean b = false 
  double d = b ? 0.0D : test(0, 0, .... 0);  // @kiszk corrected this line on July 1, 2016
  return d;
}

Exception

org.codehaus.janino.JaninoRuntimeException: org.apache.spark.sql.catalyst.expressions.GeneratedClass$SpecificSafeProjection.apply(java.lang.Object _i): Operand stack overflow at offset ...

[oontvoo]

Hi @kiszk ,

Actually, according to the JVM spec, instance methods should have no more than 254, whereas static methods 255 arguments.

The number of method parameters is limited to 255 by the definition of a method descriptor (§4.3.3), where the limit includes one unit for this in the case of instance or interface method invocations.

Why are you trying to pass so many arguments to a method?

[kiszk]

@oontvoo Yes, I know the limitation retarding the number of arguments. This example has only 127 arguments (< 254). In the original problem, a Java code generator in Apache Spark automatically generates a code regarding to a user's Spark program with a complex schema.

High operand stack height is not a problem only in a method argument. You can see another example.

double d = 1;
double r = 0 + d + d + d + ..... + d;

[oontvoo]

Fix has been merged, hence closing this issue