pmap 0.1.0 → 1.0.0

data/.travis.yml

@@ -0,0 +1,9 @@
+language: ruby
+rvm:
+  - 1.8.7
+  - 1.9.2
+  - 1.9.3
+  - jruby-18mode
+  - jruby-19mode
+  - rbx-18mode
+  - rbx-19mode

data/Gemfile

@@ -2,3 +2,7 @@ source "http://rubygems.org"
 
 # Specify your gem's dependencies in pmap.gemspec
 gemspec
+
+group :development, :test do
+  gem 'rake'
+end

data/README.md

@@ -1,4 +1,4 @@
-pmap
+pmap [![Build Status](https://secure.travis-ci.org/bruceadams/pmap.png)](http://travis-ci.org/bruceadams/pmap) [![Code Climate](https://codeclimate.com/badge.png)](https://codeclimate.com/github/bruceadams/pmap)
 ====
 
 This Ruby gem adds two methods to any Enumerable (notably including

data/Rakefile

@@ -1,2 +1,8 @@
 require 'bundler'
 Bundler::GemHelper.install_tasks
+
+task :default => [:test]
+
+task :test do
+  ruby "test/pmap_test.rb"
+end

data/lib/pmap.rb

@@ -4,21 +4,30 @@ require 'thread' unless defined?(Mutex)
 $pmap_default_thread_count ||= 64
 
 module PMap
+  class DummyOutput
+    def []=(idx)
+    end
+  end
+
   def self.included(base)
     base.class_eval do
       # Parallel "map" for any Enumerable.
       # Requires a block of code to run for each Enumerable item.
       # [thread_count] is number of threads to create. Optional.
       def pmap(thread_count=nil, &proc)
-        raise ArgumentError, "thread_count must be at least one." unless
-          thread_count.nil? or (thread_count.respond_to?(:>=) and thread_count >= 1)
-        # This seems overly fussy... (code smell)
         in_array = self.to_a        # I'm not sure how expensive this is...
+        out_array = Array.new(in_array.size)
+        process_core(thread_count, in_array, out_array, &proc)
+        out_array
+      end
+
+      def process_core(thread_count, in_array, out_array, &proc)
+        thread_count = thread_count(thread_count, in_array)
         size = in_array.size
-        thread_count = [thread_count||$pmap_default_thread_count, size].min
-        out_array = Array.new(size)
+
         semaphore = Mutex.new
         index = -1                  # Our use of index is protected by semaphore
+
         threads = (0...thread_count).map {
           Thread.new {
             i = nil
@@ -28,17 +37,22 @@ module PMap
           }
         }
         threads.each {|t| t.join}
-        out_array
       end
+      private :process_core
+
+      def thread_count(user_requested_count, items)
+        user_requested_count ||= $pmap_default_thread_count
+        raise ArgumentError, "thread_count must be at least one." unless
+          user_requested_count.respond_to?(:>=) && user_requested_count >= 1
+        [user_requested_count, items.size].min
+      end
+      private :thread_count
 
       # Parallel "each" for any Enumerable.
       # Requires a block of code to run for each Enumerable item.
       # [thread_count] is number of threads to create. Optional.
       def peach(thread_count=nil, &proc)
-        # This is doing some extra work: building a return array that is
-        # thrown away. How can I share the core code of "pmap" here and omit
-        # the output array creation?
-        pmap(thread_count, &proc)
+        process_core(thread_count, self.to_a, DummyOutput.new, &proc)
         self
       end
     end

data/lib/pmap/version.rb

@@ -1,3 +1,3 @@
 module Pmap
-  VERSION = "0.1.0"
+  VERSION = "1.0.0"
 end

data/pmap.gemspec

@@ -6,8 +6,8 @@ Gem::Specification.new do |s|
   s.name        = 'pmap'
   s.version     = Pmap::VERSION
   s.platform    = Gem::Platform::RUBY
-  s.authors     = ['Bruce Adams']
-  s.email       = ['bruce.adams@acm.org']
+  s.authors     = ['Bruce Adams', 'Jake Goulding']
+  s.email       = ['bruce.adams@acm.org', 'jake.goulding@gmail.com']
   s.homepage    = 'https://github.com/bruceadams/pmap'
   s.summary     = %q{Add parallel methods into Enumerable: pmap and peach}
   s.description = %q{Add parallel methods into Enumerable: pmap and peach}

metadata

@@ -1,30 +1,27 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: pmap
-version: !ruby/object:Gem::Version 
+version: !ruby/object:Gem::Version
+  version: 1.0.0
   prerelease: 
-  version: 0.1.0
 platform: ruby
-authors: 
+authors:
 - Bruce Adams
+- Jake Goulding
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2011-05-09 00:00:00 -04:00
-default_executable: 
+date: 2013-01-29 00:00:00.000000000 Z
 dependencies: []
-
-description: "Add parallel methods into Enumerable: pmap and peach"
-email: 
+description: ! 'Add parallel methods into Enumerable: pmap and peach'
+email:
 - bruce.adams@acm.org
+- jake.goulding@gmail.com
 executables: []
-
 extensions: []
-
 extra_rdoc_files: []
-
-files: 
+files:
 - .gitignore
+- .travis.yml
 - Gemfile
 - LICENSE
 - README.md
@@ -33,33 +30,34 @@ files:
 - lib/pmap/version.rb
 - pmap.gemspec
 - test/pmap_test.rb
-has_rdoc: true
 homepage: https://github.com/bruceadams/pmap
 licenses: []
-
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
+required_ruby_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+      segments:
+      - 0
+      hash: 2137837020685364568
+required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      version: "0"
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+      segments:
+      - 0
+      hash: 2137837020685364568
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.6.2
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
-summary: "Add parallel methods into Enumerable: pmap and peach"
+summary: ! 'Add parallel methods into Enumerable: pmap and peach'
 test_files: []
-