Merge pull request #145 from xinminlabs/awesomecode-lint-ruby-style-p…

…erlbackrefs-48052 Auto corrected by following Lint Ruby Style/PerlBackrefs
synvert-hq · Feb 21, 2021 · 3b20ea4 · 3b20ea4
2 parents cc85050 + 0f5ceee
commit 3b20ea4
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/lib/synvert/core/engine/erb.rb b/lib/synvert/core/engine/erb.rb
@@ -31,21 +31,21 @@ def decode(source)
         private
 
         def decode_ruby_stmt(source)
-          source.gsub(/#{ERUBY_STMT_SPLITTER}(.+?)#{ERUBY_STMT_SPLITTER}/m) { "<%#{$1}%>" }
+          source.gsub(/#{ERUBY_STMT_SPLITTER}(.+?)#{ERUBY_STMT_SPLITTER}/m) { "<%#{Regexp.last_match(1)}%>" }
         end
 
         def decode_ruby_output(source)
-          source.gsub(/@output_buffer.append=\((.+?)\);#{ERUBY_EXPR_SPLITTER}/m) { "<%=#{$1}%>" }.gsub(
+          source.gsub(/@output_buffer.append=\((.+?)\);#{ERUBY_EXPR_SPLITTER}/m) { "<%=#{Regexp.last_match(1)}%>" }.gsub(
             /@output_buffer.append= (.+?)\s+(do|\{)(\s*\|[^|]*\|)?\s*#{ERUBY_EXPR_SPLITTER}/m
           ) { |m| "<%=#{m.sub('@output_buffer.append= ', '').sub(ERUBY_EXPR_SPLITTER, '')}%>" }
         end
 
         def decode_html_output(source)
-          source.gsub(/@output_buffer.safe_append='(.+?)'.freeze;/m) { reverse_escape_text($1) }.gsub(
+          source.gsub(/@output_buffer.safe_append='(.+?)'.freeze;/m) { reverse_escape_text(Regexp.last_match(1)) }.gsub(
             /@output_buffer.safe_append=\((.+?)\);#{ERUBY_EXPR_SPLITTER}/m
-          ) { reverse_escape_text($1) }.gsub(
+          ) { reverse_escape_text(Regexp.last_match(1)) }.gsub(
             /@output_buffer.safe_append=(.+?)\s+(do|\{)(\s*\|[^|]*\|)?\s*#{ERUBY_EXPR_SPLITTER}/m
-          ) { reverse_escape_text($1) }
+          ) { reverse_escape_text(Regexp.last_match(1)) }
         end
 
         def remove_erubis_buf(source)
@@ -119,9 +119,9 @@ def add_stmt(src, code)
           index =
             case code
             when /\A(\s*)\r?\n/
-              $1.length
+              Regexp.last_match(1).length
             when /\A(\s+)/
-              $1.end_with?(' ') ? $1.length - 1 : $1.length
+              Regexp.last_match(1).end_with?(' ') ? Regexp.last_match(1).length - 1 : Regexp.last_match(1).length
             else
               0
             end

diff --git a/lib/synvert/core/node_ext.rb b/lib/synvert/core/node_ext.rb
@@ -364,7 +364,7 @@ def match?(rules)
     # @raise [Synvert::Core::MethodNotSupported] if string in block {{ }} does not support.
     def rewritten_source(code)
       code.gsub(/{{(.*?)}}/m) do
-        old_code = $1
+        old_code = Regexp.last_match(1)
         if respond_to? old_code.split(/\.|\[/).first
           evaluated = instance_eval old_code
           case evaluated

diff --git a/lib/synvert/core/rewriter/helper.rb b/lib/synvert/core/rewriter/helper.rb
@@ -65,7 +65,7 @@ def add_curly_brackets_if_necessary(code)
     #
     #   strip_brackets("(1..100)") #=> "1..100"
     def strip_brackets(code)
-      code.sub(/^\((.*)\)$/) { $1 }.sub(/^\[(.*)\]$/) { $1 }.sub(/^{(.*)}$/) { $1 }
+      code.sub(/^\((.*)\)$/) { Regexp.last_match(1) }.sub(/^\[(.*)\]$/) { Regexp.last_match(1) }.sub(/^{(.*)}$/) { Regexp.last_match(1) }
     end
   end
 end