From 682237be6ecabe9f35e7a7a1cb16a1eb54b69fb9 Mon Sep 17 00:00:00 2001 From: ksss Date: Mon, 21 Nov 2016 12:39:21 +0900 Subject: Reimplements Range#first Range#first shouldn't call `Range#to_a` on infinity range. --- mrbgems/mruby-range-ext/src/range.c | 27 --------------------------- 1 file changed, 27 deletions(-) (limited to 'mrbgems/mruby-range-ext/src') diff --git a/mrbgems/mruby-range-ext/src/range.c b/mrbgems/mruby-range-ext/src/range.c index 1d9312b03..32222a594 100644 --- a/mrbgems/mruby-range-ext/src/range.c +++ b/mrbgems/mruby-range-ext/src/range.c @@ -66,32 +66,6 @@ mrb_range_cover(mrb_state *mrb, mrb_value range) return mrb_false_value(); } -/* - * call-seq: - * rng.first -> obj - * rng.first(n) -> an_array - * - * Returns the first object in the range, or an array of the first +n+ - * elements. - * - * (10..20).first #=> 10 - * (10..20).first(3) #=> [10, 11, 12] - */ -static mrb_value -mrb_range_first(mrb_state *mrb, mrb_value range) -{ - mrb_int num; - mrb_value array; - struct RRange *r = mrb_range_ptr(range); - - if (mrb_get_args(mrb, "|i", &num) == 0) { - return r->edges->beg; - } - - array = mrb_funcall(mrb, range, "to_a", 0); - return mrb_funcall(mrb, array, "first", 1, mrb_fixnum_value(num)); -} - /* * call-seq: * rng.last -> obj @@ -193,7 +167,6 @@ mrb_mruby_range_ext_gem_init(mrb_state* mrb) struct RClass * s = mrb_class_get(mrb, "Range"); mrb_define_method(mrb, s, "cover?", mrb_range_cover, MRB_ARGS_REQ(1)); - mrb_define_method(mrb, s, "first", mrb_range_first, MRB_ARGS_OPT(1)); mrb_define_method(mrb, s, "last", mrb_range_last, MRB_ARGS_OPT(1)); mrb_define_method(mrb, s, "size", mrb_range_size, MRB_ARGS_NONE()); } -- cgit v1.2.3