Apply review

rubocop · Dec 28, 2020 · a57f30f · a57f30f
1 parent ccf2ef8
commit a57f30f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/config/default.yml b/config/default.yml
@@ -218,7 +218,7 @@ Performance/RedundantSortBlock:
 Performance/RedundantSplitRegexpArgument:
  Description: 'This cop identifies places where `split` argument can be replaced from a deterministic regexp to a string.'
  Enabled: pending
- VersionAdded: '1.9'
+ VersionAdded: '1.10'
 
 Performance/RedundantStringChars:
  Description: 'Checks for redundant `String#chars`.'

diff --git a/lib/rubocop/cop/performance/redundant_split_regexp_argument.rb b/lib/rubocop/cop/performance/redundant_split_regexp_argument.rb
@@ -43,9 +43,7 @@ def autocorrect(corrector, node)
  new_argument = node.first_argument.source[1..-2]
  new_argument.delete!('\\') unless STR_SPECIAL_CHARS.include?(new_argument)
 
- new_source = "#{node.receiver.source}.split(\"#{new_argument}\")"
-
- corrector.replace(node.source_range, new_source)
+ corrector.replace(node.first_argument, "\"#{new_argument}\"")
  end
  end
  end