From 3a65eb6acbac9cec3595bdd047bfbe9f43917cd4 Mon Sep 17 00:00:00 2001 From: AbishekRaj2007 Date: Thu, 6 Aug 2026 20:07:34 +0530 Subject: [PATCH] Use .public_send instead of .send for calls to public methods --- Library/Homebrew/cask/audit.rb | 2 +- Library/Homebrew/compilers/compiler_selector.rb | 2 +- Library/Homebrew/dev-cmd/bottle.rb | 8 ++++---- Library/Homebrew/dev-cmd/bump-cask-pr.rb | 4 ++-- Library/Homebrew/dev-cmd/bump.rb | 10 +++++----- Library/Homebrew/exceptions.rb | 2 +- Library/Homebrew/formula.rb | 2 +- Library/Homebrew/formulary.rb | 2 +- Library/Homebrew/livecheck/livecheck.rb | 2 +- Library/Homebrew/test_bot/test_formulae.rb | 2 +- Library/Homebrew/utils/formatter.rb | 4 ++-- 11 files changed, 20 insertions(+), 20 deletions(-) diff --git a/Library/Homebrew/cask/audit.rb b/Library/Homebrew/cask/audit.rb index b1f01bdeeb..c0d6f600ba 100644 --- a/Library/Homebrew/cask/audit.rb +++ b/Library/Homebrew/cask/audit.rb @@ -341,7 +341,7 @@ module Cask def audit_required_stanzas odebug "Auditing required stanzas" [:version, :sha256, :url, :homepage].each do |sym| - add_error "a #{sym} stanza is required" unless cask.send(sym) + add_error "a #{sym} stanza is required" unless cask.public_send(sym) end add_error "at least one name stanza is required" if cask.name.empty? # TODO: specific DSL knowledge should not be spread around in various files like this diff --git a/Library/Homebrew/compilers/compiler_selector.rb b/Library/Homebrew/compilers/compiler_selector.rb index 0359da8f65..75f6e5df08 100644 --- a/Library/Homebrew/compilers/compiler_selector.rb +++ b/Library/Homebrew/compilers/compiler_selector.rb @@ -113,7 +113,7 @@ class CompilerSelector when "gcc", GNU_GCC_REGEXP versions.gcc_version(name.to_s) else - versions.send(:"#{name}_build_version") + versions.public_send(:"#{name}_build_version") end end end diff --git a/Library/Homebrew/dev-cmd/bottle.rb b/Library/Homebrew/dev-cmd/bottle.rb index dbd420ef0c..1aa9dc7df6 100644 --- a/Library/Homebrew/dev-cmd/bottle.rb +++ b/Library/Homebrew/dev-cmd/bottle.rb @@ -199,7 +199,7 @@ module Homebrew old_keys.each do |key| next if skip_keys.include?(key) - old_value = old_bottle_spec.send(key).to_s + old_value = old_bottle_spec.public_send(key).to_s new_value = new_values[key].to_s next if old_value.present? && new_value == old_value @@ -623,14 +623,14 @@ module Homebrew old_spec = formula.bottle_specification if args.keep_old? && !old_spec.checksums.empty? mismatches = [:root_url, :rebuild].reject do |key| - old_spec.send(key) == bottle.send(key) + old_spec.public_send(key) == bottle.public_send(key) end unless mismatches.empty? bottle_path.unlink if bottle_path.exist? mismatches.map! do |key| - old_value = old_spec.send(key).inspect - value = bottle.send(key).inspect + old_value = old_spec.public_send(key).inspect + value = bottle.public_send(key).inspect "#{key}: old: #{old_value}, new: #{value}" end diff --git a/Library/Homebrew/dev-cmd/bump-cask-pr.rb b/Library/Homebrew/dev-cmd/bump-cask-pr.rb index b27325e3d8..1e02993a6f 100644 --- a/Library/Homebrew/dev-cmd/bump-cask-pr.rb +++ b/Library/Homebrew/dev-cmd/bump-cask-pr.rb @@ -265,7 +265,7 @@ module Homebrew # something that should be handled by better version replacement logic # but this is a workaround for now. arch_values = arch_values.sort_by do |type| - new_version_value = Version.new(new_version.send(type) || "0") + new_version_value = Version.new(new_version.public_send(type) || "0") Livecheck::LivecheckVersion.create(cask, new_version_value) end.reverse end @@ -303,7 +303,7 @@ module Homebrew next if unsupported_nested_arch_stanza?(contents, :version, arch) || unsupported_nested_arch_stanza?(contents, :sha256, arch) - bump_version = new_version.send(arch) || new_version.general + bump_version = new_version.public_send(arch) || new_version.general next unless bump_version version_scope = cask_stanza_scope(contents, :version, arch) diff --git a/Library/Homebrew/dev-cmd/bump.rb b/Library/Homebrew/dev-cmd/bump.rb index 0fbaf564c8..fdbe10bea4 100644 --- a/Library/Homebrew/dev-cmd/bump.rb +++ b/Library/Homebrew/dev-cmd/bump.rb @@ -588,11 +588,11 @@ module Homebrew if multiple_versions[:new] (BumpVersionParser::VERSION_SYMBOLS - [:general]).each do |arch| - new_arch_version = new_version.send(arch) + new_arch_version = new_version.public_send(arch) next if new_arch_version.blank? || message?(new_arch_version) current_arch_version = if multiple_versions[:current] - current_version.send(arch) + current_version.public_send(arch) else current_version.general end @@ -603,7 +603,7 @@ module Homebrew elsif multiple_versions[:current] if (new_version_general = new_version.general) && !message?(new_version_general) (BumpVersionParser::VERSION_SYMBOLS - [:general]).each do |arch| - current_arch_version = current_version.send(arch) + current_arch_version = current_version.public_send(arch) next if current_arch_version.blank? || new_version_general <= current_arch_version version_args << "--version-#{arch}=#{new_version_general}" @@ -629,12 +629,12 @@ module Homebrew current_versions = {} new_versions = {} BumpVersionParser::VERSION_SYMBOLS.each do |type| - current_version_value = current_version.send(type) + current_version_value = current_version.public_send(type) if current_version_value current_versions[type] = Livecheck::LivecheckVersion.create(formula_or_cask, current_version_value) end - new_version_value = new_version.send(type) + new_version_value = new_version.public_send(type) if message?(new_version_value) # Store a string, so we can easily tell when a value is a message # rather than a version diff --git a/Library/Homebrew/exceptions.rb b/Library/Homebrew/exceptions.rb index ec2f66ee7c..f448fab17a 100644 --- a/Library/Homebrew/exceptions.rb +++ b/Library/Homebrew/exceptions.rb @@ -721,7 +721,7 @@ class BuildError < RuntimeError require "diagnostic" checks = Homebrew::Diagnostic::Checks.new checks.build_error_checks.each do |check| - out = checks.send(check) + out = checks.public_send(check) next if out.nil? puts diff --git a/Library/Homebrew/formula.rb b/Library/Homebrew/formula.rb index 4a6167f4b8..e87d628813 100644 --- a/Library/Homebrew/formula.rb +++ b/Library/Homebrew/formula.rb @@ -415,7 +415,7 @@ class Formula sig { params(name: T.any(String, Symbol)).void } def spec_eval(name) - spec = self.class.send(name).dup + spec = self.class.public_send(name).dup return unless spec.url spec.owner = self diff --git a/Library/Homebrew/formulary.rb b/Library/Homebrew/formulary.rb index 7f92c0a2d4..09ea75baff 100644 --- a/Library/Homebrew/formulary.rb +++ b/Library/Homebrew/formulary.rb @@ -417,7 +417,7 @@ module Formulary if f.any_version_installed? tab = Tab.for_formula(f) resolved_spec = spec || tab.spec - f.active_spec = resolved_spec if f.send(resolved_spec) + f.active_spec = resolved_spec if f.public_send(resolved_spec) f.build = tab if f.head? && tab.tabfile k = Keg.new(T.must(tab.tabfile).parent) diff --git a/Library/Homebrew/livecheck/livecheck.rb b/Library/Homebrew/livecheck/livecheck.rb index d43d4e1300..f9b024d6b4 100644 --- a/Library/Homebrew/livecheck/livecheck.rb +++ b/Library/Homebrew/livecheck/livecheck.rb @@ -535,7 +535,7 @@ module Homebrew when :url package_or_resource.url&.to_s if package_or_resource.is_a?(Cask::Cask) || package_or_resource.is_a?(Resource) when :head, :stable - package_or_resource.send(livecheck_url)&.url if package_or_resource.is_a?(Formula) + package_or_resource.public_send(livecheck_url)&.url if package_or_resource.is_a?(Formula) when :homepage package_or_resource.homepage unless package_or_resource.is_a?(Resource) end diff --git a/Library/Homebrew/test_bot/test_formulae.rb b/Library/Homebrew/test_bot/test_formulae.rb index b98583e19a..c263cd2c83 100644 --- a/Library/Homebrew/test_bot/test_formulae.rb +++ b/Library/Homebrew/test_bot/test_formulae.rb @@ -368,7 +368,7 @@ module Homebrew sig { params(formula: Formula).returns(T::Boolean) } def downloads_using_homebrew_curl?(formula) [:stable, :head].any? do |spec_name| - next false unless (spec = formula.send(spec_name)) + next false unless (spec = formula.public_send(spec_name)) spec.using == :homebrew_curl || spec.resources.values.any? { |r| r.using == :homebrew_curl } end diff --git a/Library/Homebrew/utils/formatter.rb b/Library/Homebrew/utils/formatter.rb index f1856cb430..221aac5bae 100644 --- a/Library/Homebrew/utils/formatter.rb +++ b/Library/Homebrew/utils/formatter.rb @@ -120,11 +120,11 @@ module Formatter if prefix.nil? && color.nil? string.to_s elsif prefix.nil? - "#{Tty.send(T.must(color))}#{string}#{Tty.reset}" + "#{Tty.public_send(T.must(color))}#{string}#{Tty.reset}" elsif color.nil? "#{prefix} #{string}" else - "#{Tty.send(color)}#{prefix}#{Tty.reset} #{string}" + "#{Tty.public_send(color)}#{prefix}#{Tty.reset} #{string}" end end private_class_method :prefix