use "S!" specifier of mrb_get_args() to improve #2882 fix.

d0e67aad · Yukihiro "Matz" Matsumoto · 02474daa · d0e67aad
Commit d0e67aad authored Jul 13, 2015 by Yukihiro "Matz" Matsumoto
Show whitespace changes
Inline Side-by-side

Showing with 1 addition and 5 deletions

src/array.c src/array.c +1 -5

No files found.
--- a/src/array.c
+++ b/src/array.c
@@ -1031,11 +1031,7 @@ mrb_ary_join_m(mrb_state *mrb, mrb_value ary)
 {
  mrb_value sep = mrb_nil_value();
-  mrb_get_args(mrb, "|o", &sep);
+  mrb_get_args(mrb, "|S!", &sep);
-  if (!(mrb_nil_p(sep) || mrb_type(sep) == MRB_TT_STRING)) {
-    mrb_raise(mrb, E_TYPE_ERROR, "expected String");
-    return mrb_nil_value();
-  }
  return mrb_ary_join(mrb, ary, sep);
 }