From 108f7536b3fe9a05343a6c36541c5d96153c7f1d Mon Sep 17 00:00:00 2001 From: Nobuyoshi Nakada Date: Sat, 7 Mar 2020 16:41:19 +0900 Subject: [PATCH] Removed unnecessary `chomp` As `String#split` with the default argument drops trailing newline as a separator, preceding `String#chomp` is futile. --- lib/net/smtp.rb | 2 +- spec/mspec/tool/sync/sync-rubyspec.rb | 2 +- test/ruby/test_process.rb | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/net/smtp.rb b/lib/net/smtp.rb index 460ad08233..da8c3f26b1 100644 --- a/lib/net/smtp.rb +++ b/lib/net/smtp.rb @@ -1042,7 +1042,7 @@ module Net return {} unless @string[3, 1] == '-' h = {} @string.lines.drop(1).each do |line| - k, *v = line[4..-1].chomp.split + k, *v = line[4..-1].split h[k] = v end h diff --git a/spec/mspec/tool/sync/sync-rubyspec.rb b/spec/mspec/tool/sync/sync-rubyspec.rb index 72572e2eb2..c1218bb02c 100644 --- a/spec/mspec/tool/sync/sync-rubyspec.rb +++ b/spec/mspec/tool/sync/sync-rubyspec.rb @@ -135,7 +135,7 @@ def rebase_commits(impl) else last_merge = `git log --grep='^#{impl.last_merge_message}' -n 1 --format='%H %ct'` end - last_merge, commit_timestamp = last_merge.chomp.split(' ') + last_merge, commit_timestamp = last_merge.split(' ') raise "Could not find last merge" unless last_merge puts "Last merge is #{last_merge}" diff --git a/test/ruby/test_process.rb b/test/ruby/test_process.rb index 53599a6318..a5eb0a5844 100644 --- a/test/ruby/test_process.rb +++ b/test/ruby/test_process.rb @@ -2381,7 +2381,7 @@ EOS end w.close assert_equal "exec failed\n", r.gets - vals = r.gets.chomp.split.map!(&:to_i) + vals = r.gets.split.map!(&:to_i) assert_operator vals[0], :>, vals[1], vals.inspect _, status = Process.waitpid2(pid) end