diff options
author | normal <normal@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2015-04-10 21:05:29 +0000 |
---|---|---|
committer | normal <normal@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2015-04-10 21:05:29 +0000 |
commit | db2ff03bc0f87df5ec699d3d3d0f9c58c7d114fb (patch) | |
tree | 375602951a2e3e6638b11340996a5b86cae763e4 | |
parent | 9e459f7aa900c30500a8f04ec57a57a28f49183d (diff) | |
download | ruby-db2ff03bc0f87df5ec699d3d3d0f9c58c7d114fb.tar.gz |
lib/net/protocol.rb (rbuf_fill): avoid exception with read_nonblock
Exceptions are noisy in debug output and waste allocations.
Use "exception: false" introduced in 2.1 to return symbols for
common errors instead.
Follow-up commits will be prepared to reduce EOFError exceptions
to further quiet debug output and IO.select may be replaced by
io/wait methods if available to reduce allocations.
[ruby-core:68787] [Feature #11044]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@50219 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | lib/net/protocol.rb | 28 |
2 files changed, 18 insertions, 15 deletions
@@ -1,3 +1,8 @@ +Sat Apr 11 04:46:42 2015 Eric Wong <e@80x24.org> + + * lib/net/protocol.rb (rbuf_fill): avoid exception with read_nonblock + [ruby-core:68787] [Feature #11044] + Fri Apr 10 23:57:44 2015 Nobuyoshi Nakada <nobu@ruby-lang.org> * dir.c (need_normalization): use getattrlist() if fgetattrlist() diff --git a/lib/net/protocol.rb b/lib/net/protocol.rb index 25477014fb..5a20c5d515 100644 --- a/lib/net/protocol.rb +++ b/lib/net/protocol.rb @@ -149,23 +149,21 @@ module Net # :nodoc: BUFSIZE = 1024 * 16 def rbuf_fill - begin - @rbuf << @io.read_nonblock(BUFSIZE) - rescue IO::WaitReadable - if IO.select([@io], nil, nil, @read_timeout) - retry - else - raise Net::ReadTimeout - end - rescue IO::WaitWritable + case rv = @io.read_nonblock(BUFSIZE, exception: false) + when String + return @rbuf << rv + when :wait_readable + IO.select([@io], nil, nil, @read_timeout) or raise Net::ReadTimeout + # continue looping + when :wait_writable # OpenSSL::Buffering#read_nonblock may fail with IO::WaitWritable. # http://www.openssl.org/support/faq.html#PROG10 - if IO.select(nil, [@io], nil, @read_timeout) - retry - else - raise Net::ReadTimeout - end - end + IO.select(nil, [@io], nil, @read_timeout) or raise Net::ReadTimeout + # continue looping + when nil + # callers do not care about backtrace, so avoid allocating for it + raise EOFError, 'end of file reached', [] + end while true end def rbuf_consume(len) |