Minimal changes to avoid Fixnum deprecation warnings

Notably, this does not revise the documentation to match.

Author: matthewd

lib/concurrent/atomic/cyclic_barrier.rb

@@ -18,7 +18,7 @@ class CyclicBarrier < Synchronization::LockableObject
     #
     # @raise [ArgumentError] if `parties` is not an integer or is less than zero
     def initialize(parties, &block)
-      if !parties.is_a?(Fixnum) || parties < 1
+      if !parties.is_a?(Integer) || parties < 1
         raise ArgumentError.new('count must be in integer greater than or equal zero')
       end
       super(&nil)

lib/concurrent/atomic/mutex_atomic_fixnum.rb

@@ -77,8 +77,8 @@ def ns_set(value)
 
     # @!visibility private
     def range_check!(value)
-      if !value.is_a?(Fixnum)
-        raise ArgumentError.new('value value must be a Fixnum')
+      if !value.is_a?(Integer)
+        raise ArgumentError.new('value must be an Integer')
       elsif value > MAX_VALUE
         raise RangeError.new("#{value} is greater than the maximum value of #{MAX_VALUE}")
       elsif value < MIN_VALUE

lib/concurrent/atomic/mutex_count_down_latch.rb

@@ -9,7 +9,7 @@ class MutexCountDownLatch < Synchronization::LockableObject
 
     # @!macro count_down_latch_method_initialize
     def initialize(count = 1)
-      unless count.is_a?(Fixnum) && count >= 0
+      unless count.is_a?(Integer) && count >= 0
         raise ArgumentError.new('count must be in integer greater than or equal zero')
       end
       super()

lib/concurrent/atomic/mutex_semaphore.rb

@@ -9,7 +9,7 @@ class MutexSemaphore < Synchronization::LockableObject
 
     # @!macro semaphore_method_initialize
     def initialize(count)
-      unless count.is_a?(Fixnum) && count >= 0
+      unless count.is_a?(Integer) && count >= 0
         fail ArgumentError, 'count must be an non-negative integer'
       end
       super()
@@ -18,7 +18,7 @@ def initialize(count)
 
     # @!macro semaphore_method_acquire
     def acquire(permits = 1)
-      unless permits.is_a?(Fixnum) && permits > 0
+      unless permits.is_a?(Integer) && permits > 0
         fail ArgumentError, 'permits must be an integer greater than zero'
       end
       synchronize do
@@ -45,7 +45,7 @@ def drain_permits
 
     # @!macro semaphore_method_try_acquire
     def try_acquire(permits = 1, timeout = nil)
-      unless permits.is_a?(Fixnum) && permits > 0
+      unless permits.is_a?(Integer) && permits > 0
         fail ArgumentError, 'permits must be an integer greater than zero'
       end
       synchronize do
@@ -59,7 +59,7 @@ def try_acquire(permits = 1, timeout = nil)
 
     # @!macro semaphore_method_release
     def release(permits = 1)
-      unless permits.is_a?(Fixnum) && permits > 0
+      unless permits.is_a?(Integer) && permits > 0
         fail ArgumentError, 'permits must be an integer greater than zero'
       end
       synchronize do
@@ -81,7 +81,7 @@ def release(permits = 1)
     #
     # @!visibility private
     def reduce_permits(reduction)
-      unless reduction.is_a?(Fixnum) && reduction >= 0
+      unless reduction.is_a?(Integer) && reduction >= 0
         fail ArgumentError, 'reduction must be an non-negative integer'
       end
       synchronize { @free -= reduction }

lib/concurrent/map.rb

@@ -203,7 +203,7 @@ def marshal_load(hash)
     undef :freeze
 
     # @!visibility private
-    DEFAULT_OBJ_ID_STR_WIDTH = (2**50).class == Fixnum ? 14 : 7 # we want to look "native", 7 for 32-bit, 14 for 64-bit
+    DEFAULT_OBJ_ID_STR_WIDTH = 0.size == 4 ? 7 : 14 # we want to look "native", 7 for 32-bit, 14 for 64-bit
     # override default #inspect() method: firstly, we don't want to be spilling our guts (i-vars), secondly, MRI backend's
     # #inspect() call on its @backend i-var will bump @backend's iter level while possibly yielding GVL
     def inspect
@@ -227,8 +227,8 @@ def populate_from(hash)
     end
 
     def validate_options_hash!(options)
-      if (initial_capacity = options[:initial_capacity]) && (!initial_capacity.kind_of?(Fixnum) || initial_capacity < 0)
-        raise ArgumentError, ":initial_capacity must be a positive Fixnum"
+      if (initial_capacity = options[:initial_capacity]) && (!initial_capacity.kind_of?(Integer) || initial_capacity < 0)
+        raise ArgumentError, ":initial_capacity must be a positive Integer"
       end
       if (load_factor = options[:load_factor]) && (!load_factor.kind_of?(Numeric) || load_factor <= 0 || load_factor > 1)
         raise ArgumentError, ":load_factor must be a number between 0 and 1"

spec/concurrent/edge/future_spec.rb

@@ -202,7 +202,7 @@
       c = Concurrent.future { raise 'c' }
 
       Concurrent.zip(a, b, c).chain { |*args| q << args }
-      expect(q.pop.flatten.map(&:class)).to eq [FalseClass, Fixnum, NilClass, NilClass, NilClass, RuntimeError, RuntimeError]
+      expect(q.pop.flatten.map(&:class)).to eq [FalseClass, 1.class, NilClass, NilClass, NilClass, RuntimeError, RuntimeError]
       Concurrent.zip(a, b, c).rescue { |*args| q << args }
       expect(q.pop.map(&:class)).to eq [NilClass, RuntimeError, RuntimeError]