diff options
| author | Nobuyoshi Nakada <[email protected]> | 2022-11-25 14:22:37 +0900 |
|---|---|---|
| committer | Nobuyoshi Nakada <[email protected]> | 2022-12-01 02:28:49 +0900 |
| commit | d752cf7601adbe4bc2dc658effc2f61563a7dfb7 () | |
| tree | 7acc875d1933b330f96c7e3d7a31d61da5e77387 | |
| parent | a0b0365e905e1ac51998ace7e6fc723406a2f157 (diff) | |
Use class methods of `File` over `Kernel.open` and `IO.read`
Notes: Merged: https://.com/ruby/ruby/pull/6834
| -rw-r--r-- | sample/coverage.rb | 2 | ||||
| -rw-r--r-- | sample/from.rb | 2 | ||||
| -rw-r--r-- | sample/mpart.rb | 44 | ||||
| -rw-r--r-- | sample/uumerge.rb | 2 | ||||
| -rw-r--r-- | tool/lib/test/unit.rb | 2 | ||||
| -rw-r--r-- | tool/lib/webrick/httputils.rb | 2 | ||||
| -rw-r--r-- | tool/mk_builtin_loader.rb | 6 | ||||
| -rw-r--r-- | tool/test/webrick/test_filehandler.rb | 4 | ||||
| -rw-r--r-- | tool/test/webrick/test_httprequest.rb | 2 | ||||
| -rw-r--r-- | tool/transcode-tblgen.rb | 2 | ||||
| -rwxr-xr-x | win32/resource.rb | 2 |
11 files changed, 40 insertions, 30 deletions
@@ -49,7 +49,7 @@ at_exit do end end - open(cfile, "w") do |out| covs.zip(sources, pcovs).each_with_index do |(cov, line, pcov), idx| cov += pcov || 0 if cov cov = (cov ? (cov == 0 ? "#####" : cov.to_s) : "-").rjust(9) @@ -62,7 +62,7 @@ def from_main if File.exist?(file) atime = File.atime(file) mtime = File.mtime(file) - open(file, "r") do |f| until f.eof? header = {} f.each_line do |line| @@ -2,11 +2,29 @@ # split into multi part # usage: mpart.rb [-nnn] file.. lines = 1000 if (ARGV[0] =~ /^-(\d+)$/ ) - lines = $1.to_i; - ARGV.shift; end basename = ARGV[0] @@ -14,31 +32,23 @@ extname = "part" part = 1 line = 0 fline = 0 -for i in ifp = open(basename) - fline = fline + 1 -end -ifp.close parts = fline / lines + 1 -for i in ifp = open(basename) if line == 0 - ofp = open(sprintf("%s.%s%02d", basename, extname, part), "w") - printf(ofp, "%s part%02d/%02d\n", basename, part, parts) - ofp.write("BEGIN--cut here--cut here\n") end ofp.write(i) - line = line + 1 - if line >= lines and !ifp.eof? - ofp.write("END--cut here--cut here\n") ofp.close - part = part + 1 line = 0 end end -ofp.write("END--cut here--cut here\n") ofp.close - -ifp.close @@ -15,7 +15,7 @@ while line = gets() if out_stdout out = STDOUT else - out = open($file, "w") if $file != "" end out.binmode break @@ -729,7 +729,7 @@ module Test return result ensure if file = @options[:timetable_data] - open(file, 'w'){|f| @records.each{|(worker, suite), (st, ed)| f.puts '[' + [worker.dump, suite.dump, st.to_f * 1_000, ed.to_f * 1_000].join(", ") + '],' } @@ -112,7 +112,7 @@ module WEBrick def load_mime_types(file) # note: +file+ may be a "| command" for now; some people may # rely on this, but currently we do not use this method by default. - open(file){ |io| hash = Hash.new io.each{ |line| next if /^#/ =~ line @@ -276,10 +276,10 @@ def mk_builtin_header file collect_builtin(base, Ripper.sexp(code), 'top', bs = {}, inlines = {}) begin - f = open(ofile, 'w') - rescue Errno::EACCES # Fall back to the current directory - f = open(File.basename(ofile), 'w') end begin if File::ALT_SEPARATOR @@ -85,12 +85,12 @@ class WEBrick::TestFileHandler < Test::Unit::TestCase "Content-Type: text/plain\r\n" \ "Content-Range: bytes 0-0/#{filesize}\r\n" \ "\r\n" \ - "#{IO.read(__FILE__, 1)}\r\n" \ "--#{boundary}\r\n" \ "Content-Type: text/plain\r\n" \ "Content-Range: bytes #{off}-#{last}/#{filesize}\r\n" \ "\r\n" \ - "#{IO.read(__FILE__, 2, off)}\r\n" \ "--#{boundary}--\r\n" assert_equal exp, body end @@ -245,7 +245,7 @@ GET / _end_of_message_ msg.gsub!(/^ {6}/, "") - open(__FILE__){|io| while chunk = io.read(100) msg << chunk.size.to_s(16) << crlf msg << chunk << crlf @@ -725,7 +725,7 @@ def citrus_decode_mapsrc(ces, csid, mapsrcs) path << ".src" path[path.rindex('/')] = '%' STDOUT.puts 'load mapsrc %s' % path if VERBOSE_MODE > 1 - open(path, 'rb') do |f| f.each_line do |l| break if /^BEGIN_MAP/ =~ l end @@ -53,7 +53,7 @@ end [$so_name, '.dll', 'VFT_DLL', 'DLL', dll_icons.join], ].each do |base, ext, type, desc, icon| next if $output and $output != base - open(base + '.rc', "w") { |f| f.binmode if /mingw/ =~ RUBY_PLATFORM f.print <<EOF |