Updated CAtomicFixnum to use long long. Added max/min constants.

Author: jdantonio

ext/concurrent_ruby_ext/atomic_fixnum.c

@@ -12,7 +12,7 @@ VALUE atomic_fixnum_allocate(VALUE klass) {
 }
 
 VALUE method_atomic_fixnum_initialize(int argc, VALUE* argv, VALUE self) {
-  VALUE value = INT2NUM(0);
+  VALUE value = LL2NUM(0);
   rb_check_arity(argc, 0, 1);
   if (argc == 1) {
     Check_Type(argv[0], T_FIXNUM);
@@ -33,13 +33,13 @@ VALUE method_atomic_fixnum_value_set(VALUE self, VALUE value) {
 }
 
 VALUE method_atomic_fixnum_increment(VALUE self) {
-  long value = NUM2INT((VALUE) DATA_PTR(self));
-  return method_atomic_fixnum_value_set(self, INT2NUM(value + 1));
+  long long value = NUM2LL((VALUE) DATA_PTR(self));
+  return method_atomic_fixnum_value_set(self, LL2NUM(value + 1));
 }
 
 VALUE method_atomic_fixnum_decrement(VALUE self) {
-  long value = NUM2INT((VALUE) DATA_PTR(self));
-  return method_atomic_fixnum_value_set(self, INT2NUM(value - 1));
+  long long value = NUM2LL((VALUE) DATA_PTR(self));
+  return method_atomic_fixnum_value_set(self, LL2NUM(value - 1));
 }
 
 VALUE method_atomic_fixnum_compare_and_set(VALUE self, VALUE rb_expect, VALUE rb_update) {

ext/concurrent_ruby_ext/rb_concurrent.c

@@ -43,6 +43,8 @@ void Init_concurrent_ruby_ext() {
   rb_define_method(rb_cAtomicBoolean, "make_false", method_atomic_boolean_make_false, 0);
 
   // CAtomicFixnum
+  rb_define_const(rb_cAtomicFixnum, "MIN_VALUE", LL2NUM(LLONG_MIN));
+  rb_define_const(rb_cAtomicFixnum, "MAX_VALUE", LL2NUM(LLONG_MAX));
   rb_define_alloc_func(rb_cAtomicFixnum, atomic_fixnum_allocate);
   rb_define_method(rb_cAtomicFixnum, "initialize", method_atomic_fixnum_initialize, -1);
   rb_define_method(rb_cAtomicFixnum, "value", method_atomic_fixnum_value, 0);

lib/concurrent/atomic/atomic_fixnum.rb

@@ -10,6 +10,10 @@ module Concurrent
   #   @see http://docs.oracle.com/javase/7/docs/api/java/util/concurrent/atomic/AtomicLong.html java.util.concurrent.atomic.AtomicLong
   class MutexAtomicFixnum
 
+    # http://stackoverflow.com/questions/535721/ruby-max-integer
+    MIN_VALUE = -(2**(0.size * 8 -2))
+    MAX_VALUE = (2**(0.size * 8 -2) -1)
+
     # @!macro [attach] atomic_fixnum_method_initialize
     #
     # Creates a new `AtomicFixnum` with the given initial value.
@@ -106,6 +110,9 @@ def compare_and_set(expect, update)
     # @!macro atomic_fixnum
     class JavaAtomicFixnum
 
+      MIN_VALUE = Java::JavaLang::Long::MIN_VALUE
+      MAX_VALUE = Java::JavaLang::Long::MAX_VALUE
+
       # @!macro atomic_fixnum_method_initialize
       #
       def initialize(init = 0)