From 3547efe6386bb4ad224be49b723d16cde2067f9c Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Sat, 5 Sep 2009 19:33:33 -0700 Subject: Don't bother creating HeaderHash unless we need it It's expensive to create if not needed, and no current released version of Rack has my proposed optimizations for it yet... --- lib/clogger/pure.rb | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'lib/clogger/pure.rb') diff --git a/lib/clogger/pure.rb b/lib/clogger/pure.rb index 718db9a..2800802 100644 --- a/lib/clogger/pure.rb +++ b/lib/clogger/pure.rb @@ -14,6 +14,7 @@ class Clogger @fmt_ops = compile_format(opts[:format] || Format::Common, opts) @wrap_body = need_wrap_body?(@fmt_ops) @reentrant = nil + @need_resp = need_response_headers?(@fmt_ops) @body_bytes_sent = 0 end @@ -25,7 +26,7 @@ class Clogger raise TypeError, "app response not a 3 element Array: #{resp.inspect}" end status, headers, body = resp - headers = Rack::Utils::HeaderHash.new(headers) + headers = Rack::Utils::HeaderHash.new(headers) if @need_resp if wrap_body? @reentrant = env['rack.multithread'] @env, @status, @headers, @body = env, status, headers, body -- cgit v1.2.3-24-ge0c7