Merge pull request Homebrew#16405 from dduugg/regexp-match

Some minor regexp matching perf improvements

Library/Homebrew/cask/artifact/abstract_artifact.rb

@@ -20,7 +20,7 @@ def self.english_name
       end
 
       def self.english_article
-        @english_article ||= (english_name =~ /^[aeiou]/i) ? "an" : "a"
+        @english_article ||= /^[aeiou]/i.match?(english_name) ? "an" : "a"
       end
 
       def self.dsl_key

Library/Homebrew/cask/audit.rb

@@ -843,7 +843,7 @@ def get_repo_data(regex)
     def bad_url_format?(regex, valid_formats_array)
       return false unless cask.url.to_s.match?(regex)
 
-      valid_formats_array.none? { |format| cask.url.to_s =~ format }
+      valid_formats_array.none? { |format| cask.url.to_s.match?(format) }
     end
 
     sig { returns(T::Boolean) }

Library/Homebrew/dev-cmd/bottle.rb

@@ -294,7 +294,7 @@ def self.formula_ignores(formula)
 
     # Ignore matches to go keg, because all go binaries are statically linked.
     any_go_deps = formula.deps.any? do |dep|
-      dep.name =~ Version.formula_optionally_versioned_regex(:go)
+      Version.formula_optionally_versioned_regex(:go).match?(dep.name)
     end
     if any_go_deps
       go_regex = Version.formula_optionally_versioned_regex(:go, full: false)

Library/Homebrew/extend/os/mac/formula_cellar_checks.rb

@@ -32,7 +32,7 @@ def check_openssl_links
     keg = Keg.new(formula.prefix)
     system_openssl = keg.mach_o_files.select do |obj|
       dlls = obj.dynamically_linked_libraries
-      dlls.any? { |dll| %r{/usr/lib/lib(crypto|ssl|tls)\..*dylib}.match dll }
+      dlls.any? { |dll| %r{/usr/lib/lib(crypto|ssl|tls)\..*dylib}.match? dll }
     end
     return if system_openssl.empty?
 

Library/Homebrew/formula.rb

@@ -1328,7 +1328,7 @@ def link_overwrite?(path)
     self.class.link_overwrite_paths.any? do |p|
       p == to_check ||
         to_check.start_with?("#{p.chomp("/")}/") ||
-        to_check =~ /^#{Regexp.escape(p).gsub('\*', ".*?")}$/
+        /^#{Regexp.escape(p).gsub('\*', ".*?")}$/.match?(to_check)
     end
   end
 

Library/Homebrew/formula_auditor.rb

@@ -76,7 +76,7 @@ def audit_file
             !@versioned_formula &&
             (versioned_formulae = formula.versioned_formulae - [formula]) &&
             versioned_formulae.present?
-        versioned_aliases, unversioned_aliases = formula.aliases.partition { |a| a =~ /.@\d/ }
+        versioned_aliases, unversioned_aliases = formula.aliases.partition { |a| /.@\d/.match?(a) }
         _, last_alias_version = versioned_formulae.map(&:name).last.split("@")
 
         alias_name_major = "#{formula.name}@#{formula.version.major}"
@@ -715,7 +715,7 @@ def audit_specs
       return unless stable.url
 
       version = stable.version
-      problem "Stable: version (#{version}) is set to a string without a digit" if version.to_s !~ /\d/
+      problem "Stable: version (#{version}) is set to a string without a digit" unless /\d/.match?(version.to_s)
 
       stable_version_string = version.to_s
       if stable_version_string.start_with?("HEAD")

Library/Homebrew/formula_text_auditor.rb

@@ -16,7 +16,7 @@ def without_patch
     end
 
     def trailing_newline?
-      /\Z\n/ =~ @text
+      /\Z\n/.match?(@text)
     end
 
     def =~(other)
@@ -32,12 +32,12 @@ def to_s
     end
 
     def line_number(regex, skip = 0)
-      index = @lines.drop(skip).index { |line| line =~ regex }
+      index = @lines.drop(skip).index { |line| line.match?(regex) }
       index ? index + 1 : nil
     end
 
     def reverse_line_number(regex)
-      index = @lines.reverse.index { |line| line =~ regex }
+      index = @lines.reverse.index { |line| line.match?(regex) }
       index ? @lines.count - index : nil
     end
   end

Library/Homebrew/keg_relocate.rb

@@ -328,7 +328,7 @@ def self.text_matches_in_file(file, string, ignores, linked_libraries, formula_a
     Utils.popen_read("strings", "-t", "x", "-", file.to_s) do |io|
       until io.eof?
         str = io.readline.chomp
-        next if ignores.any? { |i| i =~ str }
+        next if ignores.any? { |i| str.match?(i) }
         next unless str.match? path_regex
 
         offset, match = str.split(" ", 2)

Library/Homebrew/livecheck/strategy/git.rb

@@ -66,7 +66,7 @@ def self.tag_info(url, regex = nil)
 
           # Isolate tag strings and filter by regex
           tags = stdout.gsub(%r{^.*\trefs/tags/|\^{}$}, "").split("\n").uniq.sort
-          tags.select! { |t| t =~ regex } if regex
+          tags.select! { |t| regex.match?(t) } if regex
           tags_data[:tags] = tags
 
           tags_data

Library/Homebrew/patch.rb

@@ -78,7 +78,7 @@ def contents
     path.open("rb") do |f|
       loop do
         line = f.gets
-        break if line.nil? || line =~ /^__END__$/
+        break if line.nil? || /^__END__$/.match?(line)
       end
       while (line = f.gets)
         data << line

Library/Homebrew/rubocops/extend/formula_cop.rb

@@ -233,7 +233,7 @@ def file_path_allowed?
         paths_to_exclude = [%r{/Library/Homebrew/test/}]
         return true if @file_path.nil? # file_path is nil when source is directly passed to the cop, e.g. in specs
 
-        @file_path !~ Regexp.union(paths_to_exclude)
+        !@file_path.match?(Regexp.union(paths_to_exclude))
       end
 
       def on_system_methods

Library/Homebrew/rubocops/keg_only.rb

@@ -34,7 +34,7 @@ def audit_formula(_node, _class_node, _parent_class_node, body_node)
           reason = string_content(reason).sub(name, "")
           first_word = reason.split.first
 
-          if reason =~ /\A[A-Z]/ && !reason.start_with?(*allowlist)
+          if /\A[A-Z]/.match?(reason) && !reason.start_with?(*allowlist)
             problem "'#{first_word}' from the `keg_only` reason should be '#{first_word.downcase}'." do |corrector|
               reason[0] = reason[0].downcase
               corrector.replace(@offensive_node.source_range, "\"#{reason}\"")

Library/Homebrew/rubocops/options.rb

@@ -26,7 +26,7 @@ def audit_formula(_node, _class_node, _parent_class_node, body_node)
             option = string_content(option)
             problem UNI_DEPRECATION_MSG if option == "universal"
 
-            if option !~ /with(out)?-/ &&
+            if !/with(out)?-/.match?(option) &&
                option != "cxx11" &&
                option != "universal"
               problem "Options should begin with with/without. " \

Library/Homebrew/sandbox.rb

@@ -172,7 +172,7 @@ def exec(*args)
       logs = Utils.popen_read("syslog", *syslog_args)
 
       # These messages are confusing and non-fatal, so don't report them.
-      logs = logs.lines.reject { |l| l.match(/^.*Python\(\d+\) deny file-write.*pyc$/) }.join
+      logs = logs.lines.grep_v(/^.*Python\(\d+\) deny file-write.*pyc$/).join
 
       unless logs.empty?
         if @logfile

Library/Homebrew/test/cask/audit_spec.rb

@@ -5,7 +5,7 @@
 describe Cask::Audit, :cask do
   def include_msg?(problems, msg)
     if msg.is_a?(Regexp)
-      Array(problems).any? { |problem| problem[:message] =~ msg }
+      Array(problems).any? { |problem| msg.match?(problem[:message]) }
     else
       Array(problems).any? { |problem| problem[:message] == msg }
     end

Library/Homebrew/utils/bottles.rb

@@ -51,7 +51,7 @@ def extname_tag_rebuild(filename)
 
       def receipt_path(bottle_file)
         bottle_file_list(bottle_file).find do |line|
-          line =~ %r{.+/.+/INSTALL_RECEIPT.json}
+          %r{.+/.+/INSTALL_RECEIPT.json}.match?(line)
         end
       end