fizx-thread_pool 0.2.0 → 0.2.1

data/CHANGELOG

@@ -1,3 +1,5 @@
+0.2.1
+- mutex in case Array#<< and Array#shift aren't atomic
 0.2.0
 - synchronous_execute
 - queue_limit

data/lib/thread_pool.rb

@@ -1,15 +1,17 @@
 # Hooray 
+require "thread"
 class ThreadPool
   class Executor
     attr_reader :active
     
-    def initialize(queue)
+    def initialize(queue, mutex)
       @thread = Thread.new do
         loop do
-          if block = queue.shift
+          mutex.synchronize { @block = queue.shift }
+          if @block
             @active = true
-            block.call
-            block.complete = true
+            @block.call
+            @block.complete = true
           else
             @active = false
             sleep 0.01
@@ -27,20 +29,25 @@ class ThreadPool
   
   # Initialize with number of threads to run
   def initialize(count, queue_limit = 0)
+    @mutex = Mutex.new
     @executors = []
     @queue = []
     @queue_limit = queue_limit
     @count = count    
-    count.times { @executors << Executor.new(@queue) }
+    count.times { @executors << Executor.new(@queue, @mutex) }
   end
   
   # Runs the block at some time in the near future
   def execute(&block)
     init_completable(block)
+    
     if @queue_limit > 0
       sleep 0.01 until @queue.size < @queue_limit
     end
-    @queue << block
+      
+    @mutex.synchronize do
+      @queue << block
+    end
   end
   
   # Runs the block at some time in the near future, and blocks until complete  

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: fizx-thread_pool
 version: !ruby/object:Gem::Version 
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors: 
 - Kyle Maxwell