thread 0.1.5 → 0.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 19359ceb19e36b411191f6a1ad561f7d1d4494b1
-  data.tar.gz: 26aa332d8178f937334fb00c01b6d52767aa0164
+  metadata.gz: 6bd6eca648d0974faadfe04e24cbd3a696f386ba
+  data.tar.gz: f0e27ddf6ed2c9acb872667b3363bb309262aa08
 SHA512:
-  metadata.gz: 53b93154104816e06c93c0b4477aa1a65ebc9c878b9e8723881ccf3ea5e2f9ab41c8454b6889018fb0132e9ece4c6efcac3731bff77a987629f4e784ce9d9bfc
-  data.tar.gz: 8cd976ebd6c34ac6b0d62facda8da94d67e9fda638f4a62a1d0693c12742ac1a2afca100d4c330de3638a4ea79155c3b08afb01ab1e2ef659365da16f74e9ff9
+  metadata.gz: 4d08f22d5c16219d8829efd68ea170d526e8451c9e2b05f9132b4bfba365c72b4aaad07b517ae14080d13dd3c16e954066fa3d528ef2740566876758c7e17617
+  data.tar.gz: aede26e7afce41766aa3be0031880df21854a02f202cacf1b080754227152e3f6c17342e7374e79052eb2ca6195fe26ac915465fcaaf01c0572d1328dd532422

data/lib/thread/pool.rb

@@ -191,20 +191,26 @@ class Thread::Pool
 
 	# Are all tasks consumed ?
 	def done?
-		@todo.empty? and @waiting == @spawned
+		@mutex.synchronize {
+			@todo.empty? and @waiting == @spawned
+		}
 	end
 
 	# Wait until all tasks are consumed. The caller will be blocked until then.
 	def wait_done
-		@done_mutex.synchronize {
-			return if done?
-			@done.wait @done_mutex
-		}
+		loop do
+			@done_mutex.synchronize {
+				return self if done?
+				@done.wait @done_mutex
+			}
+		end
 	end
 
 	# Check if there are idle workers.
 	def idle?
-		@todo.length < @waiting
+		@mutex.synchronize {
+			@todo.length < @waiting
+		}
 	end
 
 	# Process Block when there is a idle worker if not block its returns
@@ -221,7 +227,6 @@ class Thread::Pool
 		end
 
 		process *args, &block
-
 	end
 
 	# Add a task to the pool which will execute the block with the given

data/spec/thread/channel_spec.rb

@@ -6,8 +6,8 @@ describe Thread::Channel do
 		ch.send 'lol'
 		ch.send 'wut'
 
-		ch.receive.should == 'lol'
-		ch.receive.should == 'wut'
+		expect(ch.receive).to eq('lol')
+		expect(ch.receive).to eq('wut')
 	end
 
 	it 'receives with constraints properly' do
@@ -15,14 +15,14 @@ describe Thread::Channel do
 		ch.send 'lol'
 		ch.send 'wut'
 
-		ch.receive { |v| v == 'wut' }.should == 'wut'
-		ch.receive.should == 'lol'
+		expect(ch.receive { |v| v == 'wut' }).to eq('wut')
+		expect(ch.receive).to eq('lol')
 	end
 
 	it 'receives nil when using non blocking mode and the channel is empty' do
 		ch = Thread.channel
 
-		ch.receive!.should == nil
+		expect(ch.receive!).to be_nil
 	end
 
 	it 'guards sending properly' do

data/spec/thread/delay_spec.rb

@@ -6,6 +6,6 @@ describe Thread::Delay do
 			42
 		}
 
-		d.value.should == 42
+		expect(d.value).to eq(42)
 	end
 end

data/spec/thread/every_spec.rb

@@ -4,6 +4,6 @@ describe Thread::Every do
 	it 'delivers a value properly' do
 		e = Thread.every(5) { sleep 0.02; 42 }
 
-		e.value.should == 42
+		expect(e.value).to eq(42)
 	end
 end

data/spec/thread/future_spec.rb

@@ -7,7 +7,7 @@ describe Thread::Future do
 			42
 		}
 
-		f.value.should == 42
+		expect(f.value).to eq(42)
 	end
 
 	it 'properly checks if anything has been delivered' do
@@ -17,9 +17,9 @@ describe Thread::Future do
 			42
 		}
 
-		f.delivered?.should == false
+		expect(f.delivered?).to eq(false)
 		sleep 0.03
-		f.delivered?.should == true
+		expect(f.delivered?).to eq(true)
 	end
 
 	it 'does not block when a timeout is passed' do
@@ -29,6 +29,6 @@ describe Thread::Future do
 			42
 		}
 
-		f.value(0).should == nil
+		expect(f.value(0)).to be_nil
 	end
 end

data/spec/thread/pipe_spec.rb

@@ -7,17 +7,17 @@ describe Thread::Pipe do
 		p << 2
 		p << 4
 
-		p.deq.should == 16
-		p.deq.should == 32
+		expect(p.deq).to eq(16)
+		expect(p.deq).to eq(32)
 	end
 
 	it 'empty works properly' do
 		p = Thread |-> d { sleep 0.02; d * 2 } |-> d { d * 4 }
 
-		p.empty?.should == true
+		expect(p.empty?).to be(true)
 		p.enq 42
-		p.empty?.should == false
+		expect(p.empty?).to be(false)
 		p.deq
-		p.empty?.should == true
+		expect(p.empty?).to be(true)
 	end
 end

data/spec/thread/promise_spec.rb

@@ -10,7 +10,7 @@ describe Thread::Promise do
 			p << 42
 		}
 
-		p.value.should == 42
+		expect(p.value).to eq(42)
 	end
 
 	it 'properly checks if anything has been delivered' do
@@ -22,14 +22,14 @@ describe Thread::Promise do
 			p << 42
 		}
 
-		p.delivered?.should == false
+		expect(p.delivered?).to be(false)
 		sleep 0.03
-		p.delivered?.should == true
+		expect(p.delivered?).to be(true)
 	end
 
 	it 'does not block when a timeout is passed' do
 		p = Thread.promise
 
-		p.value(0).should == nil
+		expect(p.value(0)).to be(nil)
 	end
 end

data/thread.gemspec

@@ -4,7 +4,7 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 
 Gem::Specification.new do |spec|
   spec.name          = "thread"
-  spec.version       = "0.1.5"
+  spec.version       = "0.1.6"
   spec.authors       = ["meh."]
   spec.email         = ["meh@schizofreni.co"]
   spec.summary       = %q{Various extensions to the base thread library.}

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: thread
 version: !ruby/object:Gem::Version
-  version: 0.1.5
+  version: 0.1.6
 platform: ruby
 authors:
 - meh.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-02-13 00:00:00.000000000 Z
+date: 2015-04-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec