Merge pull request #73 from k0kubun/integer-unification

Don't use Fixnum and Bignum in Ruby 2.4
cookpad · Jan 19, 2017 · 8ef010e · 8ef010e
2 parents c817733 + fcf21fe
commit 8ef010e
Showing 1 changed file with 16 additions and 15 deletions.
diff --git a/lib/garage/representer.rb b/lib/garage/representer.rb
@@ -166,22 +166,23 @@ def encode_value(value, responder, selector)
       end
     end
 
+    PRIMITIVE_CLASSES = [
+      ActiveSupport::TimeWithZone,
+      Date,
+      Time,
+      Integer,
+      Float,
+      Hash,
+      Array,
+      String,
+      NilClass,
+      TrueClass,
+      FalseClass,
+      Symbol,
+    ]
+
     def primitive?(value)
-      [
-        ActiveSupport::TimeWithZone,
-        Date,
-        Time,
-        Bignum,
-        Fixnum,
-        Float,
-        Hash,
-        Array,
-        String,
-        NilClass,
-        TrueClass,
-        FalseClass,
-        Symbol,
-      ].any? {|k| value.is_a?(k) }
+      PRIMITIVE_CLASSES.any? {|k| value.is_a?(k) }
     end
 
     private