From 1faf2322b0776f7e51d4aa1f776ee4a0d268bff1 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Fri, 23 Jan 2015 02:17:20 +0000 Subject: reduce File::Stat object allocations Since we only support 1.9.3+, io.stat.size may be simplified to io.size to reduce allocations of File::Stat objects. --- test/test_output_buffering.rb | 2 +- test/test_stream_file.rb | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) (limited to 'test') diff --git a/test/test_output_buffering.rb b/test/test_output_buffering.rb index 4911b21..9e2e400 100644 --- a/test/test_output_buffering.rb +++ b/test/test_output_buffering.rb @@ -123,7 +123,7 @@ class TestOutputBuffering < Testcase def gplv3.each raise "SHOULD NOT BE CALLED" end - size = gplv3.stat.size + size = gplv3.size len = size.to_s ranges = Rack::Utils.byte_ranges(e, size) status = 200 diff --git a/test/test_stream_file.rb b/test/test_stream_file.rb index 7dbd512..54344f4 100644 --- a/test/test_stream_file.rb +++ b/test/test_stream_file.rb @@ -9,7 +9,7 @@ class TestStreamFile < Testcase def test_stream_file fp = File.open("COPYING") - sf = Yahns::StreamFile.new(fp, true, 0, fp.stat.size) + sf = Yahns::StreamFile.new(fp, true, 0, fp.size) refute sf.respond_to?(:close) sf.wbuf_close(nil) assert fp.closed? @@ -18,7 +18,7 @@ class TestStreamFile < Testcase def test_fd fp = File.open("COPYING") obj = DevFD.new("/dev/fd/#{fp.fileno}") - sf = Yahns::StreamFile.new(obj, true, 0, fp.stat.size) + sf = Yahns::StreamFile.new(obj, true, 0, fp.size) io = sf.instance_variable_get :@tmpio assert_instance_of IO, io.to_io assert_equal fp.fileno, io.fileno -- cgit v1.2.3-24-ge0c7