Merge pull request #2172 from suzukaze/refactor-numeric.c

Refactor rshift

Merge pull request #2172 from suzukaze/refactor-numeric.c
Refactor rshift
72fe54c4 · Yukihiro "Matz" Matsumoto · 85d286b4 · 129e5a6c · 72fe54c4
Commit 72fe54c4 authored May 03, 2014 by Yukihiro "Matz" Matsumoto
Show whitespace changes
Inline Side-by-side

Showing with 3 additions and 9 deletions

src/numeric.c src/numeric.c +3 -9

No files found.
--- a/src/numeric.c
+++ b/src/numeric.c
@@ -980,17 +980,11 @@ rshift(mrb_int val, mrb_int width)
  mrb_assert(width >= 0);
  if (width >= NUMERIC_SHIFT_WIDTH_MAX) {
    if (val < 0) {
-      val = -1;
-    }
-    else {
-      val = 0;
-    }
+      return mrb_fixnum_value(-1);
    }
-  else {
-    val = val >> width;
+    return mrb_fixnum_value(0);
  }
-
-  return mrb_fixnum_value(val);
+  return mrb_fixnum_value(val >> width);
 }

 static inline void