diff options
| author | Jun Hiroe <[email protected]> | 2013-06-30 02:11:37 +0900 |
|---|---|---|
| committer | Jun Hiroe <[email protected]> | 2013-06-30 02:11:37 +0900 |
| commit | a2210274eab85de9fbcf48a6509e8485482a083c (patch) | |
| tree | b3fc3ade32cda2d4664a309b345751056e2cf75a | |
| parent | dfb240627fb876ee734bed2be2938b75b9aaa9b0 (diff) | |
| download | mruby-a2210274eab85de9fbcf48a6509e8485482a083c.tar.gz mruby-a2210274eab85de9fbcf48a6509e8485482a083c.zip | |
Refactor mrb_range_eq() func in range.c.
| -rw-r--r-- | src/range.c | 30 |
1 files changed, 11 insertions, 19 deletions
diff --git a/src/range.c b/src/range.c index 7e3e03136..43de0e799 100644 --- a/src/range.c +++ b/src/range.c @@ -152,30 +152,22 @@ mrb_range_eq(mrb_state *mrb, mrb_value range) struct RRange *rr; struct RRange *ro; mrb_value obj; - mrb_bool eq_p; mrb_get_args(mrb, "o", &obj); - if (mrb_obj_equal(mrb, range, obj)) { - eq_p = 1; - } - else if (!mrb_obj_is_instance_of(mrb, obj, mrb_obj_class(mrb, range))) { /* same class? */ - eq_p = 0; - } - else { - rr = mrb_range_ptr(range); - ro = mrb_range_ptr(obj); - if (!mrb_obj_equal(mrb, rr->edges->beg, ro->edges->beg) || - !mrb_obj_equal(mrb, rr->edges->end, ro->edges->end) || - rr->excl != ro->excl) { - eq_p = 0; - } - else { - eq_p = 1; - } + if (mrb_obj_equal(mrb, range, obj)) return mrb_true_value(); + if (!mrb_obj_is_instance_of(mrb, obj, mrb_obj_class(mrb, range))) { /* same class? */ + return mrb_false_value(); } - return mrb_bool_value(eq_p); + rr = mrb_range_ptr(range); + ro = mrb_range_ptr(obj); + if (!mrb_obj_equal(mrb, rr->edges->beg, ro->edges->beg) || + !mrb_obj_equal(mrb, rr->edges->end, ro->edges->end) || + rr->excl != ro->excl) { + return mrb_false_value(); + } + return mrb_true_value(); } static int |
