about summary refs log tree commit homepage
path: root/lib/rainbows.rb
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2015-11-21 08:52:29 +0000
committerEric Wong <e@80x24.org>2015-11-21 08:55:00 +0000
commita5d88662a519406eea6f180a010af8ef829464e7 (patch)
tree77034a0a236921b1ab2c6c237ac92a016de49c10 /lib/rainbows.rb
parent9e7fb4e9b5d97cc75892c8faaea46444d438eb61 (diff)
downloadrainbows-a5d88662a519406eea6f180a010af8ef829464e7.tar.gz
unicorn 5 will only support Ruby 1.9.3 and later, so remove
some checks for Hash#compare_by_identity and IO.copy_stream
which we know exist in Ruby 1.9.

Favor &:sym proc dispatch to avoid unnecessary captures and
bytecode size increases, too.

Finally, ensure we fail fast by converting some literal
hashes to use non-arrow syntax for symbolic keys.
Diffstat (limited to 'lib/rainbows.rb')
-rw-r--r--lib/rainbows.rb5
1 files changed, 2 insertions, 3 deletions
diff --git a/lib/rainbows.rb b/lib/rainbows.rb
index f23b387..6e7e4f2 100644
--- a/lib/rainbows.rb
+++ b/lib/rainbows.rb
@@ -11,8 +11,7 @@ module Rainbows
 
   # map of numeric file descriptors to IO objects to avoid using IO.new
   # and potentially causing race conditions when using /dev/fd/
-  FD_MAP = {}
-  FD_MAP.compare_by_identity if FD_MAP.respond_to?(:compare_by_identity)
+  FD_MAP = {}.compare_by_identity
 
   require 'rainbows/const'
   require 'rainbows/http_parser'
@@ -92,7 +91,7 @@ module Rainbows
       tmp = @readers.dup
       @readers.clear
       tmp.each { |s| s.close rescue nil }.clear
-      @at_quit.each { |task| task.call }
+      @at_quit.each(&:call)
 
       # XXX hack to break out of IO.select in worker_loop for some models
       Process.kill(:QUIT, $$)