From f845a9ef76c0195254ded79c85c24332534f4057 Mon Sep 17 00:00:00 2001 From: normal Date: Mon, 21 Nov 2016 23:05:41 +0000 Subject: lib/*: remove closed checks Follow r56795. Since Ruby 2.2, calling #close on a closed socket no longer raises exceptions. * lib/cgi/session.rb (update): remove closed? check * lib/net/http.rb (finish, transport_request): ditto * lib/net/imap.rb (disconnect): ditto * lib/net/pop.rb (do_start, do_finish): ditto * lib/net/smtp.rb (do_start, do_finish): ditto * lib/open3.rb (popen_run, pipeline_run): ditto * lib/pstore.rb (transaction): ditto * lib/shell/process-controller.rb (sfork): * lib/tempfile (_close, call, Tempfile.create): ditto * lib/webrick/httpauth/htdigest.rb (flush): ditto * lib/webrick/httpauth/htpasswd.rb (flush): ditto * lib/webrick/server.rb (start_thread, cleanup_shutdown_pipe): ditto git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@56865 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- lib/webrick/httpauth/htdigest.rb | 2 +- lib/webrick/httpauth/htpasswd.rb | 2 +- lib/webrick/server.rb | 10 ++++------ 3 files changed, 6 insertions(+), 8 deletions(-) (limited to 'lib/webrick') diff --git a/lib/webrick/httpauth/htdigest.rb b/lib/webrick/httpauth/htdigest.rb index 1ef4fdb4aa..1b42c02dfa 100644 --- a/lib/webrick/httpauth/htdigest.rb +++ b/lib/webrick/httpauth/htdigest.rb @@ -79,7 +79,7 @@ module WEBrick File::rename(tmp.path, output) renamed = true ensure - tmp.close if !tmp.closed? + tmp.close File.unlink(tmp.path) if !renamed end end diff --git a/lib/webrick/httpauth/htpasswd.rb b/lib/webrick/httpauth/htpasswd.rb index f43fc2c548..8c7b09463b 100644 --- a/lib/webrick/httpauth/htpasswd.rb +++ b/lib/webrick/httpauth/htpasswd.rb @@ -84,7 +84,7 @@ module WEBrick File::rename(tmp.path, output) renamed = true ensure - tmp.close if !tmp.closed? + tmp.close File.unlink(tmp.path) if !renamed end end diff --git a/lib/webrick/server.rb b/lib/webrick/server.rb index 45bd9706d3..0a8e722b44 100644 --- a/lib/webrick/server.rb +++ b/lib/webrick/server.rb @@ -309,7 +309,7 @@ module WEBrick else @logger.debug "close:
" end - sock.close unless sock.closed? + sock.close end } end @@ -334,11 +334,9 @@ module WEBrick @shutdown_pipe = nil return if !shutdown_pipe shutdown_pipe.each {|io| - if !io.closed? - begin - io.close - rescue IOError # another thread closed io. - end + begin + io.close + rescue IOError # another thread closed io. end } end -- cgit v1.2.3