From: Krzysztof Wilczynski <krzysztof.wilczynski@linux.com>
Date: Fri, 29 Apr 2011 22:18:58 +0000 (+0100)
Subject: Small re-factor.  Changed if not to unless for code clarity.
X-Git-Url: https://gitweb.fluxo.info/?a=commitdiff_plain;h=726746649e38615b69aa372ec978b2ffa39b89d5;p=puppet-stdlib.git

Small re-factor.  Changed if not to unless for code clarity.

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

diff --git a/capitalize.rb b/capitalize.rb
index 74d4f28..ac6cc50 100644
--- a/capitalize.rb
+++ b/capitalize.rb
@@ -13,12 +13,13 @@ module Puppet::Parser::Functions
     value = arguments[0]
     klass = value.class
 
-    if not [Array, String].include?(klass)
-      raise(Puppet::ParseError, 'capitalize(): Requires either an ' +
+    unless [Array, String].include?(klass)
+      raise(Puppet::ParseError, 'capitalize(): Requires either ' +
         'array or string to work with')
     end
 
     if value.is_a?(Array)
+      # Numbers in Puppet are often string-encoded ...
       result = value.collect { |i| i.is_a?(String) ? i.capitalize : i }
     else
       result = value.capitalize