[ruby-cvs:69606] naruse:r62425 (ruby_2_5): merge revision(s) 61639: [Backport #14323]

naruse at ruby-lang.org naruse at ruby-lang.org
Fri Feb 16 17:17:36 JST 2018


naruse	2018-02-16 17:17:36 +0900 (Fri, 16 Feb 2018)

  New Revision: 62425

  https://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=62425

  Log:
    merge revision(s) 61639: [Backport #14323]
    
    net/ftp: fix FrozenError in BufferedSocket
    
    I noticed this bug while working on something else with
    RUBYOPT=-d on, existing test cases all passed with it.
    
    Note: I use String.new because it is the local style, here,
    I prefer +'' (or ''.b, for a future commit)
    
    * lib/net/ftp.rb (BufferedSocket#read): use String.new
    * test/net/ftp/test_buffered_socket.rb (test_read_nil): new test
      [ruby-core:84675] [Bug #14323]

  Modified directories:
    branches/ruby_2_5/
  Modified files:
    branches/ruby_2_5/lib/net/ftp.rb
    branches/ruby_2_5/test/net/ftp/test_buffered_socket.rb
    branches/ruby_2_5/version.h


More information about the ruby-cvs mailing list