From: Krzysztof Wilczynski <krzysztof.wilczynski@linux.com>
Date: Fri, 29 Apr 2011 23:58:37 +0000 (+0100)
Subject: Moved to unless from if not and removed TODO.
X-Git-Url: https://gitweb.fluxo.info/?a=commitdiff_plain;h=551f4ce95b5a7b141d40e654026ff3b000481096;p=puppet-stdlib.git

Moved to unless from if not and removed TODO.

Signed-off-by: Krzysztof Wilczynski <krzysztof.wilczynski@linux.com>
---

diff --git a/empty.rb b/empty.rb
index 1db9b63..e78ebf0 100644
--- a/empty.rb
+++ b/empty.rb
@@ -2,8 +2,6 @@
 # empty.rb
 #
 
-# TODO(Krzysztof Wilczynski): Support for hashes would be nice too ...
-
 module Puppet::Parser::Functions
   newfunction(:empty, :type => :rvalue, :doc => <<-EOS
     EOS
@@ -15,8 +13,8 @@ module Puppet::Parser::Functions
     value = arguments[0]
     klass = value.class
 
-    if not [Array, Hash, String].include?(klass)
-      raise(Puppet::ParseError, 'empty(): Requires either an ' +
+    unless [Array, Hash, String].include?(klass)
+      raise(Puppet::ParseError, 'empty(): Requires either ' +
         'array, hash or string to work with')
     end