From 8443c9be7df0ef4bd63f9b46894876ce0433db56 Mon Sep 17 00:00:00 2001 From: Adam Vandenberg Date: Mon, 23 Nov 2009 10:07:23 -0800 Subject: [PATCH] Some small style reformats --- Library/Homebrew/formula.rb | 14 ++++---------- bin/brew | 2 +- 2 files changed, 5 insertions(+), 11 deletions(-) diff --git a/Library/Homebrew/formula.rb b/Library/Homebrew/formula.rb index d76769bed4..ad7dbaec4a 100644 --- a/Library/Homebrew/formula.rb +++ b/Library/Homebrew/formula.rb @@ -39,9 +39,7 @@ class Formulary # Returns all formula names as strings, with or without aliases def self.names with_aliases=false everything = (HOMEBREW_REPOSITORY+'Library/Formula').children.map{|f| f.basename('.rb').to_s } - if with_aliases - everything.push *Formulary.get_aliases.keys - end + everything.push *Formulary.get_aliases.keys if with_aliases everything.sort end @@ -115,7 +113,6 @@ class Formula validate_variable :version if @version set_instance_variable 'homepage' -# raise if @homepage.nil? # not a good idea while we have eg GitManpages! CHECKSUM_TYPES.each do |type| set_instance_variable type @@ -198,7 +195,7 @@ class Formula # redefining skip_clean? in formulas is now deprecated def skip_clean? path to_check = path.relative_path_from(prefix).to_s - self.class.skip_clean_paths.include?(to_check) + self.class.skip_clean_paths.include? to_check end # yields self with current working directory set to the uncompressed tarball @@ -446,7 +443,7 @@ private end def set_instance_variable(type) - if !instance_variable_defined?("@#{type}") + unless instance_variable_defined? "@#{type}" class_value = self.class.send(type) instance_variable_set("@#{type}", class_value) if class_value end @@ -479,10 +476,7 @@ private def aka *args @aliases ||= [] - - args.each do |item| - @aliases << item.to_s - end + args.each { |item| @aliases << item.to_s } end def depends_on name, *args diff --git a/bin/brew b/bin/brew index 03acffbc63..dc0ae0ae4f 100755 --- a/bin/brew +++ b/bin/brew @@ -275,7 +275,7 @@ begin checked = {} to_check = [name] - while ! to_check.empty? + until to_check.empty? item = to_check.pop checked[item] = true