From b788a0f1eedfb358f9d57f40cec6ba0960dea3fe Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Fri, 7 Jan 2011 10:02:52 -0800 Subject: ev_core: garbage reduction with const strings Reading headers is common and we don't want to create new String objects (even if they're tiny or copy-on-write) for the GC to munch on. --- lib/rainbows/ev_core.rb | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'lib/rainbows/ev_core.rb') diff --git a/lib/rainbows/ev_core.rb b/lib/rainbows/ev_core.rb index 23505d3..a9fc41e 100644 --- a/lib/rainbows/ev_core.rb +++ b/lib/rainbows/ev_core.rb @@ -18,8 +18,8 @@ module Rainbows::EvCore # "Transfer-Encoding: chunked", and the async.callback stuff # isn't Rack::Lint-compatible, so we have to enforce it here. headers = Rack::Utils::HeaderHash.new(headers) unless Hash === headers - alive = headers.include?("Content-Length") || - !!(%r{\Achunked\z}i =~ headers["Transfer-Encoding"]) + alive = headers.include?(Content_Length) || + !!(%r{\Achunked\z}i =~ headers[Transfer_Encoding]) end write_response(status, headers, body, alive) end @@ -50,10 +50,10 @@ module Rainbows::EvCore # returns whether to enable response chunking for autochunk models def stream_response_headers(status, headers, alive) headers = Rack::Utils::HeaderHash.new(headers) - if headers['Content-Length'] + if headers[Content_Length] rv = false else - rv = !!(headers['Transfer-Encoding'] =~ %r{\Achunked\z}i) + rv = !!(headers[Transfer_Encoding] =~ %r{\Achunked\z}i) rv = false if headers.delete('X-Rainbows-Autochunk') == 'no' end write_headers(status, headers, alive) -- cgit v1.2.3-24-ge0c7