From: Krzysztof Wilczynski <krzysztof.wilczynski@linux.com>
Date: Tue, 26 Apr 2011 01:53:40 +0000 (+0100)
Subject: Small re-factor.  We prefer our local clone of the array ...
X-Git-Url: https://gitweb.fluxo.info/?a=commitdiff_plain;h=6bf04e1353b3a294aa7b7f5a0e70a5cc2743a3ee;p=puppet-stdlib.git

Small re-factor.  We prefer our local clone of the array ...

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

diff --git a/shuffle.rb b/shuffle.rb
index d92a3cd..37de723 100644
--- a/shuffle.rb
+++ b/shuffle.rb
@@ -16,19 +16,19 @@ module Puppet::Parser::Functions
       raise(Puppet::ParseError, 'shuffle(): Requires an array to work with')
     end
 
-    return []    if array.size == 0
-    return array if array.size <= 1
+    result   = array.clone
+    elements = result.size
 
-    list     = array.clone
-    elements = list.size
+    return []     if result.size == 0
+    return result if result.size <= 1
 
     # Simple implementation of Fisher–Yates in-place shuffle ...
     elements.times do |i|
       j = rand(elements - i) + i
-      list[j], list[i] = list[i], list[j]
+      result[j], result[i] = result[i], result[j]
     end
 
-    return list
+    return result
   end
 end