Ripper on_params handling of kw were using hash and not array and cra…

…shing for reqd kwargs
jruby · Jun 8, 2015 · 1d1e883 · 1d1e883
1 parent 86dfb5d
commit 1d1e883
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/core/src/main/java/org/jruby/ext/ripper/RipperParserBase.java b/core/src/main/java/org/jruby/ext/ripper/RipperParserBase.java
@@ -210,11 +210,16 @@ public IRubyObject method_optarg(IRubyObject method, IRubyObject arg) {
     }
 
     public IRubyObject keyword_arg(IRubyObject key, IRubyObject value) {
-        RubyHash hash = RubyHash.newHash(context.runtime);
+        RubyArray array = RubyArray.newArray(context.runtime, 2);
 
-        hash.fastASet(key, value);
+        array.append(key);
+        if (value != null) {
+            array.append(value);
+        } else {
+            array.append(context.nil);
+        }
 
-        return hash;
+        return array;
     }
 
     public IRubyObject new_args(IRubyObject f, IRubyObject o, IRubyObject r, IRubyObject p, ArgsTailHolder tail) {