ruby-changes:45233
From: kazu <ko1@a...>
Date: Wed, 11 Jan 2017 23:48:58 +0900 (JST)
Subject: [ruby-changes:45233] kazu:r57306 (trunk): lib/net/protocol.rb: Specify frozen_string_literal: true.
kazu 2017-01-11 23:48:51 +0900 (Wed, 11 Jan 2017) New Revision: 57306 https://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=57306 Log: lib/net/protocol.rb: Specify frozen_string_literal: true. Modified files: trunk/lib/net/protocol.rb trunk/test/net/protocol/test_protocol.rb Index: lib/net/protocol.rb =================================================================== --- lib/net/protocol.rb (revision 57305) +++ lib/net/protocol.rb (revision 57306) @@ -1,4 +1,4 @@ https://github.com/ruby/ruby/blob/trunk/lib/net/protocol.rb#L1 -# frozen_string_literal: false +# frozen_string_literal: true # # = net/protocol.rb # @@ -84,7 +84,7 @@ module Net # :nodoc: https://github.com/ruby/ruby/blob/trunk/lib/net/protocol.rb#L84 @read_timeout = read_timeout @continue_timeout = continue_timeout @debug_output = debug_output - @rbuf = '' + @rbuf = ''.dup end attr_reader :io @@ -114,7 +114,7 @@ module Net # :nodoc: https://github.com/ruby/ruby/blob/trunk/lib/net/protocol.rb#L114 public - def read(len, dest = '', ignore_eof = false) + def read(len, dest = ''.dup, ignore_eof = false) LOG "reading #{len} bytes..." read_bytes = 0 begin @@ -132,7 +132,7 @@ module Net # :nodoc: https://github.com/ruby/ruby/blob/trunk/lib/net/protocol.rb#L132 dest end - def read_all(dest = '') + def read_all(dest = ''.dup) LOG 'reading all...' read_bytes = 0 begin @@ -331,7 +331,7 @@ module Net # :nodoc: https://github.com/ruby/ruby/blob/trunk/lib/net/protocol.rb#L331 end def using_each_crlf_line - @wbuf = '' + @wbuf = ''.dup yield if not @wbuf.empty? # unterminated last line write0 dot_stuff(@wbuf.chomp) + "\r\n" Index: test/net/protocol/test_protocol.rb =================================================================== --- test/net/protocol/test_protocol.rb (revision 57305) +++ test/net/protocol/test_protocol.rb (revision 57306) @@ -1,4 +1,4 @@ https://github.com/ruby/ruby/blob/trunk/test/net/protocol/test_protocol.rb#L1 -# frozen_string_literal: false +# frozen_string_literal: true require "test/unit" require "net/protocol" require "stringio" @@ -6,7 +6,7 @@ require "stringio" https://github.com/ruby/ruby/blob/trunk/test/net/protocol/test_protocol.rb#L6 class TestProtocol < Test::Unit::TestCase def test_should_properly_dot_stuff_period_with_no_endline bug9627 = '[ruby-core:61441] [Bug #9627]' - sio = StringIO.new("") + sio = StringIO.new("".dup) imio = Net::InternetMessageIO.new(sio) email = "To: bob@a...\nlook, a period with no endline\n." imio.write_message(email) @@ -15,12 +15,12 @@ class TestProtocol < Test::Unit::TestCas https://github.com/ruby/ruby/blob/trunk/test/net/protocol/test_protocol.rb#L15 def test_each_crlf_line assert_output('', '') do - sio = StringIO.new("") + sio = StringIO.new("".dup) imio = Net::InternetMessageIO.new(sio) assert_equal(23, imio.write_message("\u3042\r\u3044\n\u3046\r\n\u3048")) assert_equal("\u3042\r\n\u3044\r\n\u3046\r\n\u3048\r\n.\r\n", sio.string) - sio = StringIO.new("") + sio = StringIO.new("".dup) imio = Net::InternetMessageIO.new(sio) assert_equal(8, imio.write_message("\u3042\r")) assert_equal("\u3042\r\n.\r\n", sio.string) -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/