admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

[Library/Homebrew/cli/named_args.rb] to_formulae_to_casks(): Reassociate its

Browse Source 
`rescue` block with the right enclosing scope.

     PR #9398 accidentally removed some semantically significant indentation in
this function, thus inadvertently kicking the `rescue` block inside out to func-
tion scope, when it moved its code further up in the file.  This may well not
confuse a Ruby interpreter, but it could trip human readers up.  Compensate and
correct the overlooked offending oversight and potential refactoring mistake by
restoring only that indentation, leaving the function in its new location.

Refs #9398.
This commit is contained in:
Bryce Glover 2020-12-05 13:57:41 -05:00
parent f901b57bb5
commit c237223e15
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
Library/Homebrew/cli/named_args.rb
View File

@ -48,8 +48,8 @@ module Homebrew
@to_formulae_and_casks ||= {} @to_formulae_and_casks ||= {}
@to_formulae_and_casks[only] ||= downcased_unique_named.flat_map do |name| @to_formulae_and_casks[only] ||= downcased_unique_named.flat_map do |name|
load_formula_or_cask(name, only: only, method: method) load_formula_or_cask(name, only: only, method: method)
rescue NoSuchKegError, FormulaUnavailableError, Cask::CaskUnavailableError rescue NoSuchKegError, FormulaUnavailableError, Cask::CaskUnavailableError
ignore_unavailable ? [] : raise ignore_unavailable ? [] : raise
end.uniq.freeze end.uniq.freeze
end end