You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@spamassassin.apache.org by mm...@apache.org on 2007/09/22 00:41:28 UTC

svn commit: r578308 - /spamassassin/trunk/lib/Mail/SpamAssassin/Util.pm

Author: mmartinec
Date: Fri Sep 21 15:41:27 2007
New Revision: 578308

URL: http://svn.apache.org/viewvc?rev=578308&view=rev
Log:
actually, the last iteration in fisher_yates_shuffle is redundant

Modified:
    spamassassin/trunk/lib/Mail/SpamAssassin/Util.pm

Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Util.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/Util.pm?rev=578308&r1=578307&r2=578308&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Util.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Util.pm Fri Sep 21 15:41:27 2007
@@ -1598,7 +1598,7 @@
 
 sub fisher_yates_shuffle {
   my ($deck) = @_;
-  for (my $i = $#{$deck}; $i >= 0; $i--) {
+  for (my $i = $#{$deck}; $i > 0; $i--) {
     my $j = int rand($i+1);
     @$deck[$i,$j] = @$deck[$j,$i];
   }