From b12cc1eb0acc0daea2e355c450aa8b9dbf5b2743 Mon Sep 17 00:00:00 2001
From: Adam Vandenberg
Date: Tue, 10 Aug 2010 11:52:03 -0700
Subject: brew audit - most audits shouldn't happen against __END__

---
 Library/Contributions/examples/brew-audit.rb | 20 +++++++++++---------
 1 file changed, 11 insertions(+), 9 deletions(-)

(limited to 'Library')

diff --git a/Library/Contributions/examples/brew-audit.rb b/Library/Contributions/examples/brew-audit.rb
index 7e28e99d5..84334a95b 100755
--- a/Library/Contributions/examples/brew-audit.rb
+++ b/Library/Contributions/examples/brew-audit.rb
@@ -52,14 +52,8 @@ def audit_formula_text text
     problems << " * md5 is empty"
   end
 
-  # DATA with no __END__
-  if (text =~ /\bDATA\b/) and not (text =~ /^\s*__END__\s*$/)
-    problems << " * 'DATA' was found, but no '__END__'"
-  end
-
-  # Don't complain about spaces in patches
-  split_patch = (text.split("__END__")[0]).strip()
-  if split_patch =~ /[ ]+$/
+  # No trailing whitespace, please
+  if text =~ /[ ]+$/
     problems << " * Trailing whitespace was found."
   end
 
@@ -79,7 +73,15 @@ def audit_some_formulae
     text = ""
     File.open(f.path, "r") { |afile| text = afile.read }
 
-    problems += audit_formula_text(text)
+    # DATA with no __END__
+    if (text =~ /\bDATA\b/) and not (text =~ /^\s*__END__\s*$/)
+      problems << " * 'DATA' was found, but no '__END__'"
+    end
+
+    # Don't try remaining audits on text in __END__
+    text_without_patch = (text.split("__END__")[0]).strip()
+
+    problems += audit_formula_text(text_without_patch)
 
     unless problems.empty?
       puts "#{f.name}:"
-- 
cgit v1.2.3