Merge pull request #46 from jdantonio/jruby-improvemnts

Code readability improvement

Author: lucasallan

lib/concurrent/atomic.rb

@@ -109,7 +109,7 @@ def initialize(init = 0)
       allocate_storage(init)
     end
 
-    if defined? java.util
+    if RUBY_PLATFORM == 'java'
       include JavaAtomicFixnum
     else
       include MutexAtomicFixnum

lib/concurrent/cached_thread_pool.rb

@@ -2,35 +2,35 @@
 
 module Concurrent
 
-  if defined? java.util
+  if RUBY_PLATFORM == 'java'
     require 'concurrent/java_cached_thread_pool'
     # @!macro [attach] cached_thread_pool
     #   A thread pool that dynamically grows and shrinks to fit the current workload.
     #   New threads are created as needed, existing threads are reused, and threads
     #   that remain idle for too long are killed and removed from the pool. These
     #   pools are particularly suited to applications that perform a high volume of
     #   short-lived tasks.
-    #  
+    #
     #   On creation a +CachedThreadPool+ has zero running threads. New threads are
     #   created on the pool as new operations are +#post+. The size of the pool
     #   will grow until +#max_length+ threads are in the pool or until the number
     #   of threads exceeds the number of running and pending operations. When a new
     #   operation is post to the pool the first available idle thread will be tasked
     #   with the new operation.
-    #  
+    #
     #   Should a thread crash for any reason the thread will immediately be removed
     #   from the pool. Similarly, threads which remain idle for an extended period
     #   of time will be killed and reclaimed. Thus these thread pools are very
     #   efficient at reclaiming unused resources.
-    #  
+    #
     #   The API and behavior of this class are based on Java's +CachedThreadPool+
     #
     #   @note When running on the JVM (JRuby) this class will inherit from +JavaCachedThreadPool+.
     #     On all other platforms it will inherit from +RubyCachedThreadPool+.
     #
     #   @see Concurrent::RubyCachedThreadPool
     #   @see Concurrent::JavaCachedThreadPool
-    #  
+    #
     #   @see http://docs.oracle.com/javase/tutorial/essential/concurrency/pools.html
     #   @see http://docs.oracle.com/javase/7/docs/api/java/util/concurrent/Executors.html
     #   @see http://docs.oracle.com/javase/8/docs/api/java/util/concurrent/ExecutorService.html

lib/concurrent/fixed_thread_pool.rb

@@ -2,7 +2,7 @@
 
 module Concurrent
 
-  if defined? java.util
+  if RUBY_PLATFORM == 'java'
     require 'concurrent/java_fixed_thread_pool'
     # @!macro [attach] fixed_thread_pool
     #
@@ -11,15 +11,15 @@ module Concurrent
     #   tasks +#post+ to the thread pool are enqueued until a thread becomes available.
     #   Should a thread crash for any reason the thread will immediately be removed
     #   from the pool and replaced.
-    #  
+    #
     #   The API and behavior of this class are based on Java's +FixedThreadPool+
     #
     #   @note When running on the JVM (JRuby) this class will inherit from +JavaFixedThreadPool+.
     #     On all other platforms it will inherit from +RubyFixedThreadPool+.
     #
     #   @see Concurrent::RubyFixedThreadPool
     #   @see Concurrent::JavaFixedThreadPool
-    #  
+    #
     #   @see http://docs.oracle.com/javase/tutorial/essential/concurrency/pools.html
     #   @see http://docs.oracle.com/javase/7/docs/api/java/util/concurrent/Executors.html
     #   @see http://docs.oracle.com/javase/8/docs/api/java/util/concurrent/ExecutorService.html

lib/concurrent/java_cached_thread_pool.rb

@@ -1,4 +1,4 @@
-if defined? java.util
+if RUBY_PLATFORM == 'java'
 
   require 'concurrent/java_thread_pool_executor'
 

lib/concurrent/java_fixed_thread_pool.rb

@@ -1,4 +1,4 @@
-if defined? java.util
+if RUBY_PLATFORM == 'java'
 
   require 'concurrent/java_thread_pool_executor'
 

lib/concurrent/java_thread_pool_executor.rb

@@ -1,4 +1,4 @@
-if defined? java.util
+if RUBY_PLATFORM == 'java'
 
   module Concurrent
 

lib/concurrent/processor_count.rb

@@ -34,7 +34,7 @@ module Concurrent
   # @see http://docs.oracle.com/javase/6/docs/api/java/lang/Runtime.html#availableProcessors()
   # @see http://msdn.microsoft.com/en-us/library/aa394373(v=vs.85).aspx
   def processor_count
-    if defined? java.lang
+    if RUBY_PLATFORM == 'java'
       java.lang.Runtime.getRuntime.availableProcessors
     else
       @@processor_count ||= begin

lib/concurrent/thread_local_var.rb

@@ -75,7 +75,7 @@ class ThreadLocalVar
 
     NIL_SENTINEL = Object.new
 
-    if defined? java.lang
+    if RUBY_PLATFORM == 'java'
       include ThreadLocalJavaStorage
     elsif Thread.current.respond_to?(:thread_variable_set)
       include ThreadLocalNewStorage

lib/concurrent/thread_pool_executor.rb

@@ -2,20 +2,20 @@
 
 module Concurrent
 
-  if defined? java.util
+  if RUBY_PLATFORM == 'java'
     require 'concurrent/java_thread_pool_executor'
     # @!macro [attach] thread_pool_executor
     #
     #   A thread pool...
-    #  
+    #
     #   The API and behavior of this class are based on Java's +ThreadPoolExecutor+
     #
     #   @note When running on the JVM (JRuby) this class will inherit from +JavaThreadPoolExecutor+.
     #     On all other platforms it will inherit from +RubyThreadPoolExecutor+.
     #
     #   @see Concurrent::RubyThreadPoolExecutor
     #   @see Concurrent::JavaThreadPoolExecutor
-    #  
+    #
     #   @see http://docs.oracle.com/javase/tutorial/essential/concurrency/pools.html
     #   @see http://docs.oracle.com/javase/7/docs/api/java/util/concurrent/Executors.html
     #   @see http://docs.oracle.com/javase/8/docs/api/java/util/concurrent/ExecutorService.html