Restored old AtomicBoolean behaviour and fixed failing tests

Author: mighe

lib/concurrent/atomic/atomic_boolean.rb

@@ -75,26 +75,24 @@ def false?
     #
     #   Explicitly sets the value to true.
     #
-    #   @return [Boolean] true if the value changed
+    #   @return [nil]
     def make_true
       @mutex.lock
-      old = @value
       @value = true
       @mutex.unlock
-      !old
+      nil
     end
 
     # @!macro [attach] atomic_boolean_method_make_false
     #
     #   Explicitly sets the value to false.
     #
-    #   @return [Boolean] true if the value changed
+    #   @return [nil]
     def make_false
       @mutex.lock
-      old = @value
       @value = false
       @mutex.unlock
-      old
+      nil
     end
   end
 
@@ -123,12 +121,12 @@ def value=(value)
 
       # @!macro [attach] atomic_boolean_method_is_true
       def true?
-        @atomic.get == true
+        @atomic.get
       end
 
       # @!macro [attach] atomic_boolean_method_is_false
       def false?
-        @atomic.get != true
+        !@atomic.get
       end
 
       # @!macro atomic_boolean_method_make_true

spec/concurrent/atomic/atomic_boolean_spec.rb

@@ -75,30 +75,30 @@
 
   describe '#make_true' do
 
-    it 'makes a false value true and returns true' do
+    it 'makes a false value true and returns nil' do
       subject = described_class.new(false)
-      subject.make_true.should be_true
+      subject.make_true.should be_nil
       subject.value.should be_true
     end
 
-    it 'keeps a true value true and returns false' do
+    it 'keeps a true value true and returns nil' do
       subject = described_class.new(true)
-      subject.make_true.should be_false
+      subject.make_true.should be_nil
       subject.value.should be_true
     end
   end
 
   describe '#make_false' do
 
-    it 'makes a true value false and returns true' do
+    it 'makes a true value false and returns nil' do
       subject = described_class.new(true)
-      subject.make_false.should be_true
+      subject.make_false.should be_nil
       subject.value.should be_false
     end
 
-    it 'keeps a false value false and returns false' do
+    it 'keeps a false value false and returns nil' do
       subject = described_class.new(false)
-      subject.make_false.should be_false
+      subject.make_false.should be_nil
       subject.value.should be_false
     end
   end