From: Simon Tatham <anakin@pobox.com>
Date: Mon, 17 Oct 2005 18:32:24 +0000 (+0000)
Subject: Just noticed a longhand shuffling operation which I must have missed
X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ian/git?a=commitdiff_plain;h=f9168d6003042b3dfa175175f54b512c5886ac18;p=sgt-puzzles.git

Just noticed a longhand shuffling operation which I must have missed
when I converted them all into calls to shuffle().

[originally from svn r6404]
---

diff --git a/net.c b/net.c
index 29a3a27..f8811a7 100644
--- a/net.c
+++ b/net.c
@@ -897,14 +897,7 @@ static void perturb(int w, int h, unsigned char *tiles, int wrapping,
     perim2 = snewn(nperim, struct xyd);
     memcpy(perim2, perimeter, nperim * sizeof(struct xyd));
     /* Shuffle the perimeter, so as to search it without directional bias. */
-    for (i = nperim; --i ;) {
-	int j = random_upto(rs, i+1);
-	struct xyd t;
-
-	t = perim2[j];
-	perim2[j] = perim2[i];
-	perim2[i] = t;
-    }
+    shuffle(perim2, nperim, sizeof(*perim2), rs);
     for (i = 0; i < nperim; i++) {
 	int x2, y2;