barney 0.10.0 → 0.10.1

data/ChangeLog

@@ -1,5 +1,15 @@
 2011-04-23  Robert Gleeson <rob@flowof.info>
 
+	* lib/barney.rb README.md ChangeLog:
+		Release 0.10.1
+
+	* lib/barney/share.rb:
+		Fix a bug where Barney::Share#history could return unordered/incorrect data 
+		when subprocesses are spawned in parallel.
+
+	* lib/barney/share.rb: 
+		Release 0.10.0
+
 	* lib/barney/share.rb: 
 		Fix a bug where Barney::Share#share could return nil.
 		If there were no duplicates in @variables, `nil` would be returned.

data/README.md

@@ -84,12 +84,12 @@ Documentation
 **API**  
 
 * [master (git)](http://rubydoc.info/github/robgleeson/barney/master/)
+* [0.10.1](http://rubydoc.info/gems/barney/0.10.1/)  
 * [0.10.0](http://rubydoc.info/gems/barney/0.10.0/)
 * [0.9.1](http://rubydoc.info/gems/barney/0.9.1/)
 * [0.9.0](http://rubydoc.info/gems/barney/0.9.0/)
 * [0.8.1](http://rubydoc.info/gems/barney/0.8.1/)
 * [0.8.0](http://rubydoc.info/gems/barney/0.8.0/)
-* [0.7.0](http://rubydoc.info/gems/barney/0.7.0)
 * …
 
 

data/lib/barney/share.rb

@@ -77,18 +77,22 @@ module Barney
     # @return [Fixnum]        Returns the Process ID(PID) of the spawned child process.  
     def fork &blk
       raise ArgumentError, "A block or Proc object is expected" unless block_given?
-      spawn_pipes
+
+      tmp_streams = Array.new @variables.size do |index|
+        StreamPair.new @variables[index], *IO.pipe
+      end
 
       @context = blk.binding
       @pid = Kernel.fork do
         blk.call
-        @streams.each do |stream|
+        tmp_streams.each do |stream|
           stream.in.close  
           stream.out.write Marshal.dump(eval("#{stream.variable}", @context))
           stream.out.close
         end
       end
 
+      @streams.push *tmp_streams
       @pid
     end
 
@@ -110,17 +114,6 @@ module Barney
     end
     alias_method :sync, :synchronize
 
-    private
-
-    # Manages the creation of pipes used for cross-process communcation.
-    # @return [void] 
-    # @api private
-    def spawn_pipes
-      @variables.each do |variable| 
-        @streams.push StreamPair.new(variable, *IO.pipe)
-      end
-    end
-
   end
 
 end

data/lib/barney.rb

@@ -2,7 +2,7 @@ require 'barney/share'
 
 module Barney 
 
-  VERSION = '0.10.0'
+  VERSION = '0.10.1'
   @proxy  = Barney::Share.new
 
   class << self

data/test/suite/lib/barney/share#fork.rb

@@ -13,7 +13,29 @@ describe Barney::Share do
         Barney::Share.new.fork
       end
     end
-    
+
+    it 'should not cause #history to return bad/incorrect data.' do
+      50.times do 
+        str = ""
+        pids = []
+
+        obj = Barney::Share.new
+        obj.share :str
+
+        %w(r u b y).each do |letter|
+          pids << obj.fork do 
+            str << letter
+          end
+        end
+
+        pids.each { |pid| Process.wait pid }
+        obj.sync
+
+        str = obj.history.map(&:value).join
+        assert_equal "ruby", str  
+      end
+    end
+      
   end
 
 end

metadata

@@ -2,7 +2,7 @@
 name: barney
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.10.0
+  version: 0.10.1
 platform: ruby
 authors: 
 - Robert Gleeson
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-04-25 00:00:00 +01:00
+date: 2011-05-03 00:00:00 +01:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency