aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorilovezfs2016-10-03 02:41:17 -0700
committerMike McQuaid2017-02-20 14:14:03 +0000
commit4c922a089bf2bb8ff379291c90d46b91978dbbb4 (patch)
tree8a0aab6acccc5a9b552ea7e31e75e9d13f261a1a
parent14ef15f591795c517b7771c5eb01863b2b577a46 (diff)
downloadbrew-4c922a089bf2bb8ff379291c90d46b91978dbbb4.tar.bz2
remove unless empty since each is no-op for empty
-rw-r--r--Library/Homebrew/keg.rb16
1 files changed, 6 insertions, 10 deletions
diff --git a/Library/Homebrew/keg.rb b/Library/Homebrew/keg.rb
index bebc3c887..06ac68310 100644
--- a/Library/Homebrew/keg.rb
+++ b/Library/Homebrew/keg.rb
@@ -239,10 +239,8 @@ class Keg
def remove_opt_record
opt_record.unlink
- unless aliases.empty?
- aliases.each do |a|
- (opt_record.parent/a).unlink
- end
+ aliases.each do |a|
+ (opt_record.parent/a).unlink
end
opt_record.parent.rmdir_if_possible
end
@@ -478,12 +476,10 @@ class Keg
def optlink(mode = OpenStruct.new)
opt_record.delete if opt_record.symlink? || opt_record.exist?
make_relative_symlink(opt_record, path, mode)
- unless aliases.empty?
- aliases.each do |a|
- alias_opt_record = opt_record.parent/a
- alias_opt_record.delete if alias_opt_record.symlink? || alias_opt_record.exist?
- make_relative_symlink(alias_opt_record, opt_record, mode)
- end
+ aliases.each do |a|
+ alias_opt_record = opt_record.parent/a
+ alias_opt_record.delete if alias_opt_record.symlink? || alias_opt_record.exist?
+ make_relative_symlink(alias_opt_record, opt_record, mode)
end
return unless oldname_opt_record