Revert only temporarily because this breaks something in Rails 4 involing Array. Figure out for 1.7.17

Author: enebo

core/src/main/java/org/jruby/RubyArray.java

@@ -3542,7 +3542,7 @@ public IRubyObject product19(ThreadContext context, IRubyObject[] args, Block bl
         int counters[] = new int[n];
 
         arrays[0] = this;
-        for (int i = 1; i < n; i++) arrays[i] = TypeConverter.to_ary(context, args[i - 1]);
+        for (int i = 1; i < n; i++) arrays[i] = args[i - 1].convertToArray();
 
         int resultLen = 1;
         for (int i = 0; i < n; i++) {

core/src/main/java/org/jruby/RubyKernel.java

@@ -375,15 +375,9 @@ public static IRubyObject abort(ThreadContext context, IRubyObject recv, IRubyOb
         return runtime.getNil(); // not reached
     }
 
-    @JRubyMethod(name = "Array", required = 1, module = true, visibility = PRIVATE, compat = RUBY1_8)
-    public static IRubyObject new_array18(ThreadContext context, IRubyObject recv, IRubyObject object) {
-        return Helpers.asArray18(context, object);
-    }
-
-    // MRI: rb_f_array
-    @JRubyMethod(name = "Array", required = 1, module = true, visibility = PRIVATE, compat = RUBY1_9)
+    @JRubyMethod(name = "Array", required = 1, module = true, visibility = PRIVATE)
     public static IRubyObject new_array(ThreadContext context, IRubyObject recv, IRubyObject object) {
-        return TypeConverter.rb_Array(context, object);
+        return Helpers.asArray18(context, object);
     }
 
     @JRubyMethod(name = "Complex", module = true, visibility = PRIVATE, compat = RUBY1_9)

core/src/main/java/org/jruby/runtime/Helpers.java

@@ -1852,7 +1852,16 @@ public static RubyArray asArray18(ThreadContext context, IRubyObject value) {
     // mri: rb_Array
     // FIXME: Replace arrayValue/asArray18 with this on 9k (currently dead -- respond_to? logic broken further down the line -- fix that first)
     public static RubyArray asArray(ThreadContext context, IRubyObject value) {
-        return TypeConverter.rb_Array(context, value);
+        RubyClass array = context.runtime.getArray();
+        IRubyObject tmp = TypeConverter.convertToTypeWithCheck19(value, array, "to_ary");
+        
+        if (tmp.isNil()) {
+            tmp = TypeConverter.convertToTypeWithCheck19(value, array, "to_a");
+
+            if (tmp.isNil()) return context.runtime.newEmptyArray();
+        }
+        
+        return (RubyArray) tmp; // converters will guarantee it is RubyArray or raise.
     }
 
     public static IRubyObject aryToAry(IRubyObject value) {

core/src/main/java/org/jruby/util/TypeConverter.java

@@ -27,7 +27,6 @@
 package org.jruby.util;
 
 import org.jruby.Ruby;
-import org.jruby.RubyArray;
 import org.jruby.RubyBignum;
 import org.jruby.RubyBoolean;
 import org.jruby.RubyClass;
@@ -284,11 +283,6 @@ public static IRubyObject checkStringType(Ruby runtime, IRubyObject obj) {
         return TypeConverter.convertToTypeWithCheck(obj, runtime.getHash(), "to_str");
     }
 
-    // MRI: rb_check_array_type
-    public static IRubyObject checkArrayType(IRubyObject self) {
-        return TypeConverter.convertToTypeWithCheck19(self, self.getRuntime().getArray(), "to_ary");
-    }
-
     public static IRubyObject handleUncoercibleObject(boolean raise, IRubyObject obj, RubyClass target) throws RaiseException {
         if (raise) throw obj.getRuntime().newTypeError("can't convert " + typeAsString(obj) + " into " + target);
 
@@ -336,24 +330,6 @@ public static IRubyObject convertToInteger(ThreadContext context, IRubyObject va
         return tmp;
     }
 
-    // MRI: rb_Array
-    public static RubyArray rb_Array(ThreadContext context, IRubyObject val) {
-        IRubyObject tmp = checkArrayType(val);
-
-        if (tmp.isNil()) {
-            tmp = convertToTypeWithCheck19(val, context.runtime.getArray(), "to_a");
-            if (tmp.isNil()) {
-                return context.runtime.newArray(val);
-            }
-        }
-        return (RubyArray)tmp;
-    }
-
-    // MRI: to_ary
-    public static RubyArray to_ary(ThreadContext context, IRubyObject ary) {
-        return (RubyArray)convertToType19(ary, context.runtime.getArray(), "to_ary");
-    }
-
     private static void raiseIntegerBaseError(ThreadContext context) {
         throw context.runtime.newArgumentError("base specified for non string value");
     }