Need not to call mrb_obj_as_string() is sep is nil.

6db38c37 · Yukihiro "Matz" Matsumoto · ce869752 · 6db38c37
Commit 6db38c37 authored Mar 18, 2017 by Yukihiro "Matz" Matsumoto
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

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

No files found.
--- a/src/array.c
+++ b/src/array.c
@@ -1071,7 +1071,9 @@ join_ary(mrb_state *mrb, mrb_value ary, mrb_value sep, mrb_value list)
 MRB_API mrb_value
 mrb_ary_join(mrb_state *mrb, mrb_value ary, mrb_value sep)
 {
-  sep = mrb_obj_as_string(mrb, sep);
+  if (!mrb_nil_p(sep)) {
+    sep = mrb_obj_as_string(mrb, sep);
+  }
  return join_ary(mrb, ary, sep, mrb_ary_new(mrb));
 }