Remove useless T.unsafe wrappers
This commit is contained in:
parent
a88397096d
commit
f3a8241e69
@ -95,7 +95,7 @@ module Homebrew
|
|||||||
.reject { |line| line.start_with?("#") || line.blank? }
|
.reject { |line| line.start_with?("#") || line.blank? }
|
||||||
.map(&:strip)
|
.map(&:strip)
|
||||||
|
|
||||||
named_args = T.unsafe(CLI::NamedArgs).new(*names, parent: args)
|
named_args = CLI::NamedArgs.new(*names, parent: args)
|
||||||
named_args.to_formulae_and_casks(ignore_unavailable: true)
|
named_args.to_formulae_and_casks(ignore_unavailable: true)
|
||||||
rescue Errno::ENOENT => e
|
rescue Errno::ENOENT => e
|
||||||
onoe e
|
onoe e
|
||||||
|
|||||||
@ -1152,7 +1152,7 @@ class Formula
|
|||||||
ENV.activate_extensions!
|
ENV.activate_extensions!
|
||||||
|
|
||||||
etc_var_dirs = [bottle_prefix/"etc", bottle_prefix/"var"]
|
etc_var_dirs = [bottle_prefix/"etc", bottle_prefix/"var"]
|
||||||
T.unsafe(Find).find(*etc_var_dirs.select(&:directory?)) do |path|
|
Find.find(*etc_var_dirs.select(&:directory?)) do |path|
|
||||||
path = Pathname.new(path)
|
path = Pathname.new(path)
|
||||||
path.extend(InstallRenamed)
|
path.extend(InstallRenamed)
|
||||||
path.cp_path_sub(bottle_prefix, HOMEBREW_PREFIX)
|
path.cp_path_sub(bottle_prefix, HOMEBREW_PREFIX)
|
||||||
@ -2672,7 +2672,7 @@ class Formula
|
|||||||
out.close
|
out.close
|
||||||
args.map!(&:to_s)
|
args.map!(&:to_s)
|
||||||
begin
|
begin
|
||||||
T.unsafe(Kernel).exec(cmd, *args)
|
Kernel.exec(cmd, *args)
|
||||||
rescue
|
rescue
|
||||||
nil
|
nil
|
||||||
end
|
end
|
||||||
|
|||||||
@ -99,7 +99,7 @@ module Homebrew
|
|||||||
def self.find_versions(url:, regex: nil, **unused, &block)
|
def self.find_versions(url:, regex: nil, **unused, &block)
|
||||||
generated = generate_input_values(url)
|
generated = generate_input_values(url)
|
||||||
|
|
||||||
T.unsafe(PageMatch).find_versions(url: generated[:url], regex: regex || generated[:regex], **unused, &block)
|
PageMatch.find_versions(url: generated[:url], regex: regex || generated[:regex], **unused, &block)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@ -102,7 +102,7 @@ module Homebrew
|
|||||||
def self.find_versions(url:, regex: nil, **unused, &block)
|
def self.find_versions(url:, regex: nil, **unused, &block)
|
||||||
generated = generate_input_values(url)
|
generated = generate_input_values(url)
|
||||||
|
|
||||||
T.unsafe(PageMatch).find_versions(url: generated[:url], regex: regex || generated[:regex], **unused, &block)
|
PageMatch.find_versions(url: generated[:url], regex: regex || generated[:regex], **unused, &block)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@ -86,7 +86,7 @@ module Homebrew
|
|||||||
def self.find_versions(url:, regex: nil, **unused, &block)
|
def self.find_versions(url:, regex: nil, **unused, &block)
|
||||||
generated = generate_input_values(url)
|
generated = generate_input_values(url)
|
||||||
|
|
||||||
T.unsafe(PageMatch).find_versions(url: generated[:url], regex: regex || generated[:regex], **unused, &block)
|
PageMatch.find_versions(url: generated[:url], regex: regex || generated[:regex], **unused, &block)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@ -54,7 +54,7 @@ module Homebrew
|
|||||||
"#{Utils.demodulize(T.must(name))} only supports a regex when using a `strategy` block"
|
"#{Utils.demodulize(T.must(name))} only supports a regex when using a `strategy` block"
|
||||||
end
|
end
|
||||||
|
|
||||||
T.unsafe(Yaml).find_versions(
|
Yaml.find_versions(
|
||||||
url: url,
|
url: url,
|
||||||
regex: regex,
|
regex: regex,
|
||||||
provided_content: provided_content,
|
provided_content: provided_content,
|
||||||
|
|||||||
@ -98,7 +98,7 @@ module Homebrew
|
|||||||
def self.find_versions(url:, regex: nil, **unused, &block)
|
def self.find_versions(url:, regex: nil, **unused, &block)
|
||||||
generated = generate_input_values(url)
|
generated = generate_input_values(url)
|
||||||
|
|
||||||
T.unsafe(PageMatch).find_versions(url: generated[:url], regex: regex || DEFAULT_REGEX, **unused, &block)
|
PageMatch.find_versions(url: generated[:url], regex: regex || DEFAULT_REGEX, **unused, &block)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@ -88,7 +88,7 @@ module Homebrew
|
|||||||
def self.find_versions(url:, regex: nil, **unused, &block)
|
def self.find_versions(url:, regex: nil, **unused, &block)
|
||||||
generated = generate_input_values(url)
|
generated = generate_input_values(url)
|
||||||
|
|
||||||
version_data = T.unsafe(PageMatch).find_versions(
|
version_data = PageMatch.find_versions(
|
||||||
url: generated[:url],
|
url: generated[:url],
|
||||||
regex: regex || generated[:regex],
|
regex: regex || generated[:regex],
|
||||||
**unused,
|
**unused,
|
||||||
|
|||||||
@ -98,7 +98,7 @@ module Homebrew
|
|||||||
def self.find_versions(url:, regex: nil, **unused, &block)
|
def self.find_versions(url:, regex: nil, **unused, &block)
|
||||||
generated = generate_input_values(url)
|
generated = generate_input_values(url)
|
||||||
|
|
||||||
T.unsafe(PageMatch).find_versions(url: generated[:url], regex: regex || generated[:regex], **unused, &block)
|
PageMatch.find_versions(url: generated[:url], regex: regex || generated[:regex], **unused, &block)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@ -84,7 +84,7 @@ module Homebrew
|
|||||||
def self.find_versions(url:, regex: nil, **unused, &block)
|
def self.find_versions(url:, regex: nil, **unused, &block)
|
||||||
generated = generate_input_values(url)
|
generated = generate_input_values(url)
|
||||||
|
|
||||||
T.unsafe(PageMatch).find_versions(url: generated[:url], regex: regex || generated[:regex], **unused, &block)
|
PageMatch.find_versions(url: generated[:url], regex: regex || generated[:regex], **unused, &block)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@ -81,7 +81,7 @@ module Homebrew
|
|||||||
def self.find_versions(url:, regex: nil, **unused, &block)
|
def self.find_versions(url:, regex: nil, **unused, &block)
|
||||||
generated = generate_input_values(url)
|
generated = generate_input_values(url)
|
||||||
|
|
||||||
T.unsafe(PageMatch).find_versions(url: generated[:url], regex: regex || DEFAULT_REGEX, **unused, &block)
|
PageMatch.find_versions(url: generated[:url], regex: regex || DEFAULT_REGEX, **unused, &block)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@ -79,7 +79,7 @@ module Homebrew
|
|||||||
def self.find_versions(url:, regex: nil, **unused, &block)
|
def self.find_versions(url:, regex: nil, **unused, &block)
|
||||||
generated = generate_input_values(url)
|
generated = generate_input_values(url)
|
||||||
|
|
||||||
T.unsafe(PageMatch).find_versions(url: generated[:url], regex: regex || generated[:regex], **unused, &block)
|
PageMatch.find_versions(url: generated[:url], regex: regex || generated[:regex], **unused, &block)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@ -93,7 +93,7 @@ module Homebrew
|
|||||||
def self.find_versions(url:, regex: nil, **unused, &block)
|
def self.find_versions(url:, regex: nil, **unused, &block)
|
||||||
generated = generate_input_values(url)
|
generated = generate_input_values(url)
|
||||||
|
|
||||||
T.unsafe(PageMatch).find_versions(url: generated[:url], regex: regex || generated[:regex], **unused, &block)
|
PageMatch.find_versions(url: generated[:url], regex: regex || generated[:regex], **unused, &block)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@ -98,7 +98,7 @@ module Homebrew
|
|||||||
def self.find_versions(url:, regex: nil, **unused, &block)
|
def self.find_versions(url:, regex: nil, **unused, &block)
|
||||||
generated = generate_input_values(url)
|
generated = generate_input_values(url)
|
||||||
|
|
||||||
T.unsafe(PageMatch).find_versions(
|
PageMatch.find_versions(
|
||||||
url: generated[:url] || url,
|
url: generated[:url] || url,
|
||||||
regex: regex || generated[:regex],
|
regex: regex || generated[:regex],
|
||||||
**unused,
|
**unused,
|
||||||
|
|||||||
@ -123,7 +123,7 @@ module Homebrew
|
|||||||
|
|
||||||
# Use the cached page content to avoid duplicate fetches
|
# Use the cached page content to avoid duplicate fetches
|
||||||
cached_content = @page_data[generated_url]
|
cached_content = @page_data[generated_url]
|
||||||
match_data = T.unsafe(PageMatch).find_versions(
|
match_data = PageMatch.find_versions(
|
||||||
url: generated_url,
|
url: generated_url,
|
||||||
regex: regex || generated[:regex],
|
regex: regex || generated[:regex],
|
||||||
provided_content: cached_content,
|
provided_content: cached_content,
|
||||||
|
|||||||
@ -101,7 +101,7 @@ class Sandbox
|
|||||||
begin
|
begin
|
||||||
command = [SANDBOX_EXEC, "-f", seatbelt.path, *args]
|
command = [SANDBOX_EXEC, "-f", seatbelt.path, *args]
|
||||||
# Start sandbox in a pseudoterminal to prevent access of the parent terminal.
|
# Start sandbox in a pseudoterminal to prevent access of the parent terminal.
|
||||||
T.unsafe(PTY).spawn(*command) do |r, w, pid|
|
PTY.spawn(*command) do |r, w, pid|
|
||||||
# Set the PTY's window size to match the parent terminal.
|
# Set the PTY's window size to match the parent terminal.
|
||||||
# Some formula tests are sensitive to the terminal size and fail if this is not set.
|
# Some formula tests are sensitive to the terminal size and fail if this is not set.
|
||||||
winch = proc do |_sig|
|
winch = proc do |_sig|
|
||||||
|
|||||||
@ -199,7 +199,7 @@ class SystemCommand
|
|||||||
|
|
||||||
pid = T.let(nil, T.nilable(Integer))
|
pid = T.let(nil, T.nilable(Integer))
|
||||||
raw_stdin, raw_stdout, raw_stderr, raw_wait_thr = ignore_interrupts do
|
raw_stdin, raw_stdout, raw_stderr, raw_wait_thr = ignore_interrupts do
|
||||||
T.unsafe(Open3).popen3(env, [executable, executable], *args, **options)
|
Open3.popen3(env, [executable, executable], *args, **options)
|
||||||
.tap { |*, wait_thr| pid = wait_thr.pid }
|
.tap { |*, wait_thr| pid = wait_thr.pid }
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user