reentrant_mutex 1.1.0 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4b626086142c3356ccc40d96a806e18f256778da
-  data.tar.gz: 1b3a1dab63094ecca1ec176f0fbf42fcd060a484
+  metadata.gz: a33897ff9e7cea3a5cbcfa698bb12641ad024c50
+  data.tar.gz: ec2ce7e6d7f82c7c38b2cfb47ffd986218db4775
 SHA512:
-  metadata.gz: 8fe2383a5392af05e16a85d14df729e323da501c64dd85789f63e56ca2798272f66ea9bbf61eee179abd16911b628dea3d3dd8a0d35c07ac6bb3eb98a742e212
-  data.tar.gz: d049c9ed3f9eaa4286d79b8a41de3c5fd4e6de075d4fe835e38e010bcbee141df586e7c611bf4467789957bcdfb6cc9c4f48c8cd89013dfe968dd807bb97ab61
+  metadata.gz: fb58a48b1aea6294cc9cfe7522207c062b5692031a1c0556a430b72393d00cc80b8f111a84e29c4280e744f6d4e262f2f2afad471ca2445e3f2a2143d0a8ba6e
+  data.tar.gz: 2576e05e015ff280ab5ad6e3dace4ef376a0720883e23cf410a6b81034ddffc2ed3a5f775ce89c639f573866295cc61151a5ed4b38b95c4fe357f4a1c90f216a

data/lib/reentrant_mutex.rb

@@ -9,11 +9,12 @@ class ReentrantMutex < Mutex
   def synchronize
     raise ThreadError, 'Must be called with a block' unless block_given?
 
-    lock
-    ret = yield
-    unlock
-
-    ret
+    begin
+      lock
+      yield
+    ensure
+      unlock
+    end
   end
 
   def lock

data/reentrant_mutex.gemspec

@@ -1,7 +1,7 @@
 # coding: utf-8
 Gem::Specification.new do |spec|
   spec.name          = 'reentrant_mutex'
-  spec.version       = '1.1.0'
+  spec.version       = '1.1.1'
   spec.authors       = ['Boris Bera']
   spec.email         = ['bboris@rsoft.ca']
   spec.summary       = %q{A simple reentrant/recursive mutex}

data/spec/reentrant_mutex_spec.rb

@@ -20,6 +20,18 @@ describe ReentrantMutex do
     it 'should yield the block' do
       expect{|b| mutex.synchronize &b}.to yield_with_no_args
     end
+
+    it 'should leave the mutex unlocked' do
+      mutex.synchronize {}
+
+      expect(mutex.locked?).to be_false
+    end
+
+    it 'should be locked in the block' do
+      mutex.synchronize do
+        expect(mutex.locked?).to be_true
+      end
+    end
   end
 
   describe '#lock' do
@@ -27,6 +39,12 @@ describe ReentrantMutex do
       mutex.lock
       expect{mutex.lock}.not_to raise_error
     end
+
+    it 'should lock the mutex' do
+      mutex.lock
+
+      expect(mutex.locked?).to be_true
+    end
   end
 
   describe '#unlock' do
@@ -41,5 +59,12 @@ describe ReentrantMutex do
       expect{mutex.unlock}.not_to raise_error
       expect{mutex.unlock}.not_to raise_error
     end
+
+    it 'should unlock the mutex' do
+      mutex.lock
+      mutex.unlock
+
+      expect(mutex.locked?).to be_false
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: reentrant_mutex
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.1
 platform: ruby
 authors:
 - Boris Bera