Renamed method for better readability.

Author: jdantonio

lib/concurrent/agent.rb

@@ -65,7 +65,7 @@ def initialize(initial, opts = {})
       @validator = Proc.new { |result| true }
       @timeout = opts.fetch(:timeout, TIMEOUT).freeze
       @observers = CopyOnWriteObserverSet.new
-      @executor = get_executor_from_options(opts)
+      @executor = get_executor_from(opts)
       init_mutex
       set_deref_options(opts)
     end

lib/concurrent/configuration.rb

@@ -65,7 +65,7 @@ def finalize_executor(executor)
 
   module OptionsParser
 
-    def get_executor_from_options(opts = {})
+    def get_executor_from(opts = {})
       if opts.has_key?(:executor)
         opts[:executor]
       elsif opts[:operation] == true || opts[:task] == false

lib/concurrent/future.rb

@@ -64,7 +64,7 @@ def initialize(opts = {}, &block)
       super(IVar::NO_VALUE, opts)
       @state = :unscheduled
       @task = block
-      @executor = get_executor_from_options(opts)
+      @executor = get_executor_from(opts)
     end
 
     # Execute an +:unscheduled+ +Future+. Immediately sets the state to +:pending+ and

lib/concurrent/promise.rb

@@ -34,7 +34,7 @@ class Promise
     def initialize(opts = {}, &block)
       opts.delete_if {|k, v| v.nil?}
 
-      @executor = get_executor_from_options(opts)
+      @executor = get_executor_from(opts)
       @parent = opts.fetch(:parent) { nil }
       @on_fulfill = opts.fetch(:on_fulfill) { Proc.new{ |result| result } }
       @on_reject = opts.fetch(:on_reject) { Proc.new{ |reason| raise reason } }

spec/concurrent/configuration_spec.rb

@@ -15,7 +15,7 @@ module Concurrent
 
   describe OptionsParser do
 
-    context 'get_executor_from_options' do
+    context 'get_executor_from' do
       pending
     end
   end