mruby
diff --git a/‎mrbgems/mruby-regexp/mrblib/string_regexp.rb‎
Lines changed: 6 additions & 8 deletions b/‎mrbgems/mruby-regexp/mrblib/string_regexp.rb‎
Lines changed: 6 additions & 8 deletions
diff --git a/‎mrbgems/mruby-regexp/test/regexp.rb‎
Lines changed: 6 additions & 2 deletions b/‎mrbgems/mruby-regexp/test/regexp.rb‎
Lines changed: 6 additions & 2 deletions
@@ -133,15 +133,13 @@ def split(pattern = nil, *args)
 
     limit_given = args.length > 0
     limit = limit_given ? args[0] : 0
+    # `__to_int` is `mrb_ensure_integer_type()`, which asks the object nothing.
+    # mruby has no implicit conversion protocol in core, so `Array.new(obj)`,
+    # `ary[obj]` and `"s" * obj` all reject an object that only defines
+    # `to_int`; dispatching it here would leave this the one place in the tree
+    # that accepts one, as the same reasoning keeps `match` off `to_str`.
     if limit_given && !limit.is_a?(Integer)
-      if limit.respond_to?(:to_int)
-        limit = limit.to_int
-        unless limit.is_a?(Integer)
-          raise TypeError, "no implicit conversion of #{limit.class} to Integer)"
-        end
-      else
-        limit = limit.__to_int
-      end
+      limit = limit.__to_int
     end
     # `nil?` and `is_a?` are redefinable, so an argument answering either one
     # could steer itself around the check below and reach `__split` instead.
 
@@ -1069,12 +1069,16 @@ class StringMatchHelperOverride < String
   assert_raise(TypeError) { "a,b".split(/,/, nil) }
   assert_equal ["a,b"], "a,b".split(/,/, 1.5)
 
+  # mruby has no implicit conversion protocol, so an object defining `to_int`
+  # is rejected here exactly as `Array.new(obj)` and `ary[obj]` reject it. The
+  # limit is never asked what it responds to, so an object overriding
+  # `respond_to?` reaches the same TypeError rather than a NoMethodError.
   limit = Object.new
   def limit.to_int; 2; end
-  assert_equal ["a", "b"], "a,b".split(/,/, limit)
+  assert_raise(TypeError) { "a,b".split(/,/, limit) }
 
   limit = Object.new
-  def limit.to_int; 1.5; end
+  def limit.respond_to?(name, include_all = false); true; end
   assert_raise(TypeError) { "a,b".split(/,/, limit) }
 end