Merge pull request #4561 from etehtsea/fix-method-params

enebo · web-flow · commit e70cf6626e7c · 2017-04-18T11:03:35.000-05:00
Fix Method#parameters for a method generated by respond_to_missing?
diff --git a/core/src/main/java/org/jruby/RubyModule.java b/core/src/main/java/org/jruby/RubyModule.java
@@ -1847,6 +1847,7 @@ public static class RespondToMissingMethod extends JavaMethod.JavaMethodNBlock {
         public RespondToMissingMethod(RubyModule implClass, Visibility vis, String methodName) {
             super(implClass, vis);
 
+            setParameterList(REST);
             site = new FunctionalCachingCallSite(methodName);
         }
         @Override
diff --git a/core/src/main/java/org/jruby/internal/runtime/methods/JavaMethod.java b/core/src/main/java/org/jruby/internal/runtime/methods/JavaMethod.java
@@ -53,6 +53,7 @@ public abstract class JavaMethod extends DynamicMethod implements Cloneable, Met
     private static final String[] ONE_REQ = new String[] { "q" };
     private static final String[] TWO_REQ = new String[] { "q", "q" };
     private static final String[] THREE_REQ = new String[] { "q", "q", "q" };
+    protected static final String[] REST = new String[] { "r" };
 
     public static final Class[][] METHODS = {
         {JavaMethodZero.class, JavaMethodZeroOrOne.class, JavaMethodZeroOrOneOrTwo.class, JavaMethodZeroOrOneOrTwoOrThree.class},
diff --git a/spec/tags/ruby/core/method/parameters_tags.txt b/spec/tags/ruby/core/method/parameters_tags.txt

Original file line number	Diff line number	Diff line change
`@@ -1847,6 +1847,7 @@ public static class RespondToMissingMethod extends JavaMethod.JavaMethodNBlock {`
`1847`	`1847`	`public RespondToMissingMethod(RubyModule implClass, Visibility vis, String methodName) {`
`1848`	`1848`	`super(implClass, vis);`
`1849`	`1849`
	`1850`	`+ setParameterList(REST);`
`1850`	`1851`	`site = new FunctionalCachingCallSite(methodName);`
`1851`	`1852`	`}`
`1852`	`1853`	`@Override`