From b80940e1d8540eb2af009f2f805b4f2055a23c5f Mon Sep 17 00:00:00 2001 From: Carlo Cabrera <30379873+carlocab@users.noreply.github.com> Date: Thu, 27 Jul 2023 12:00:10 +0800 Subject: [PATCH] os/mac/mach: simplify We can avoid having to define a local variable and then reference it inside these blocks by passing a proc instead. --- Library/Homebrew/os/mac/mach.rb | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/Library/Homebrew/os/mac/mach.rb b/Library/Homebrew/os/mac/mach.rb index 38f52dc4c9..bbd8e7c01e 100644 --- a/Library/Homebrew/os/mac/mach.rb +++ b/Library/Homebrew/os/mac/mach.rb @@ -89,18 +89,15 @@ module MachOShim def dynamically_linked_libraries(except: :none, resolve_variable_references: true) lcs = macho.dylib_load_commands.reject { |lc| lc.type == except } - names = lcs.map(&:name).map(&:to_s).uniq - - names.map! { |name| resolve_variable_name(name) } if resolve_variable_references + names.map!(&method(:resolve_variable_name)) if resolve_variable_references names end def rpaths(resolve_variable_references: true) names = macho.rpaths - - names.map! { |name| resolve_variable_name(name) } if resolve_variable_references + names.map!(&method(:resolve_variable_name)) if resolve_variable_references names end