diff options
author | Eric Wong <normalperson@yhbt.net> | 2010-07-19 10:09:50 +0000 |
---|---|---|
committer | Eric Wong <normalperson@yhbt.net> | 2010-07-19 17:04:18 -0700 |
commit | c587c585e6d327c1646ce33fa10fbef6c27fc74c (patch) | |
tree | 41f410744078577326051a9b301ce91d29355de4 | |
parent | fa70dc9090eab3b518d56daf3f968d07a4a05526 (diff) | |
download | rainbows-c587c585e6d327c1646ce33fa10fbef6c27fc74c.tar.gz |
IO#read always returns a binary string buffer if passed an explicit length to read, and we always do that. This is a small garbage reduction.
-rw-r--r-- | lib/rainbows/dev_fd_response.rb | 2 | ||||
-rw-r--r-- | lib/rainbows/response/body.rb | 2 | ||||
-rw-r--r-- | lib/rainbows/sendfile.rb | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/lib/rainbows/dev_fd_response.rb b/lib/rainbows/dev_fd_response.rb index b2367d7..ba1fdaa 100644 --- a/lib/rainbows/dev_fd_response.rb +++ b/lib/rainbows/dev_fd_response.rb @@ -31,7 +31,7 @@ class Rainbows::DevFdResponse < Struct.new(:app) return response if STATUS_WITH_NO_ENTITY_BODY.include?(status) io = body.to_io if body.respond_to?(:to_io) - io ||= File.open(body.to_path, 'rb') if body.respond_to?(:to_path) + io ||= File.open(body.to_path) if body.respond_to?(:to_path) return response if io.nil? headers = HeaderHash.new(headers) diff --git a/lib/rainbows/response/body.rb b/lib/rainbows/response/body.rb index e399df7..0a2bb5d 100644 --- a/lib/rainbows/response/body.rb +++ b/lib/rainbows/response/body.rb @@ -41,7 +41,7 @@ module Rainbows::Response::Body # :nodoc: # try to take advantage of Rainbows::DevFdResponse, calling File.open # is a last resort path = body.to_path - path =~ %r{\A/dev/fd/(\d+)\z} ? IO.new($1.to_i) : File.open(path, 'rb') + path =~ %r{\A/dev/fd/(\d+)\z} ? IO.new($1.to_i) : File.open(path) end end diff --git a/lib/rainbows/sendfile.rb b/lib/rainbows/sendfile.rb index c2e63be..2804217 100644 --- a/lib/rainbows/sendfile.rb +++ b/lib/rainbows/sendfile.rb @@ -65,7 +65,7 @@ class Rainbows::Sendfile < Struct.new(:app) # fallback in case our +to_path+ doesn't get handled for whatever reason def each(&block) buf = '' - File.open(to_path, 'rb') do |fp| + File.open(to_path) do |fp| yield buf while fp.read(0x4000, buf) end end |