diff options
| author | ilovezfs | 2016-10-03 02:41:17 -0700 | 
|---|---|---|
| committer | Mike McQuaid | 2017-02-20 14:14:03 +0000 | 
| commit | 4c922a089bf2bb8ff379291c90d46b91978dbbb4 (patch) | |
| tree | 8a0aab6acccc5a9b552ea7e31e75e9d13f261a1a | |
| parent | 14ef15f591795c517b7771c5eb01863b2b577a46 (diff) | |
| download | brew-4c922a089bf2bb8ff379291c90d46b91978dbbb4.tar.bz2 | |
remove unless empty since each is no-op for empty
| -rw-r--r-- | Library/Homebrew/keg.rb | 16 | 
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  | 
