diff options
author | Krzysztof Wilczynski <krzysztof.wilczynski@linux.com> | 2011-04-29 23:30:32 +0100 |
---|---|---|
committer | Krzysztof Wilczynski <krzysztof.wilczynski@linux.com> | 2011-04-29 23:30:32 +0100 |
commit | 0ff8b00a64426acf96d0659d1cbc1f67bba842f4 (patch) | |
tree | 34c846c222a34df5059e618be48a3585d9a78458 /chomp.rb | |
parent | 726746649e38615b69aa372ec978b2ffa39b89d5 (diff) | |
download | puppet-stdlib-0ff8b00a64426acf96d0659d1cbc1f67bba842f4.tar.gz puppet-stdlib-0ff8b00a64426acf96d0659d1cbc1f67bba842f4.tar.bz2 |
Small re-factor. Changed if not to unless for code clarity.
Signed-off-by: Krzysztof Wilczynski <krzysztof.wilczynski@linux.com>
Diffstat (limited to 'chomp.rb')
-rw-r--r-- | chomp.rb | 5 |
1 files changed, 3 insertions, 2 deletions
@@ -13,12 +13,13 @@ module Puppet::Parser::Functions value = arguments[0] klass = value.class - if not [Array, String].include?(klass) - raise(Puppet::ParseError, 'chomp(): Requires either an ' + + unless [Array, String].include?(klass) + raise(Puppet::ParseError, 'chomp(): 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.chomp : i } else result = value.chomp |