Make our HttpRequest object a global constant

This should be faster/cheaper than using an instance variable
since it's accessed in a critical code path.  Unicorn was never
designed to be reentrant or thread-safe at all, either.

1 files changed, 3 insertions, 1 deletions

diff --git a/lib/unicorn/http_request.rb b/lib/unicorn/http_request.rb
index 368305f..d7078a3 100644
--- a/lib/unicorn/http_request.rb
+++ b/lib/unicorn/http_request.rb
@@ -12,6 +12,8 @@ module Unicorn
   # 
   class HttpRequest
 
+    attr_accessor :logger
+
     # default parameters we merge into the request env for Rack handlers
     DEFAULTS = {
       "rack.errors" => $stderr,
@@ -36,7 +38,7 @@ module Unicorn
     PARSER = HttpParser.new
     PARAMS = Hash.new
 
-    def initialize(logger)
+    def initialize(logger = Configurator::DEFAULT_LOGGER)
       @logger = logger
     end