From b8de4e4af43c9850210e16ddbf4650feb194246e Mon Sep 17 00:00:00 2001
From: Luke Kanies <luke@madstop.com>
Date: Tue, 13 May 2008 21:23:22 -0500
Subject: Simplifying Confine a bit

---
 lib/facter/confine.rb | 20 ++++----------------
 1 file changed, 4 insertions(+), 16 deletions(-)

diff --git a/lib/facter/confine.rb b/lib/facter/confine.rb
index 4d30630..529c968 100644
--- a/lib/facter/confine.rb
+++ b/lib/facter/confine.rb
@@ -25,29 +25,17 @@ class Facter::Confine
 
     # Evaluate the fact, returning true or false.
     def true?
-        fact = nil
         unless fact = Facter[@fact]
             Facter.debug "No fact for %s" % @fact
             return false
         end
         value = fact.value
 
-        if value.nil?
-            return false
-        end
+        return false if value.nil?
 
-        retval = @values.find { |v|
-            if value.downcase == v.downcase
-                break true
-            end
+        @values.each { |v|
+            return true if value.downcase == v.downcase
         }
-
-        if retval
-            retval = true
-        else
-            retval = false
-        end
-
-        return retval || false
+        return false
     end
 end
-- 
cgit