shithub: puzzles

Info  •  Files  •  Log  •  Branches

Move a debug statement at the end of new_clues from the caller (just
after return) to the callee (just before). Might print something
useful in the soak-test context (where that debug statement will now
be printed and previously wasn't), but the main aim is to remove the
variable 'ngen' at the main call site, which triggered a set-but-not-
used warning if the debug statement that printed it was compiled out.

[originally from svn r9392]

--- a/pearl.c

+++ b/pearl.c

@@ -1335,6 +1335,8 @@

 	break;			       /* got it */

     }

+    debug(("%d %dx%d loops before finished puzzle.\n", ngen, w, h));

+

     return ngen;

 }

@@ -1343,14 +1345,12 @@

 {

     char *grid, *clues;

     char *desc;

-    int ngen, w = params->w, h = params->h, i, j;

+    int w = params->w, h = params->h, i, j;

     grid = snewn(w*h, char);

     clues = snewn(w*h, char);

-    ngen = new_clues(params, rs, clues, grid);

-

-    debug(("%d %dx%d loops before finished puzzle.\n", ngen, w, h));

+    new_clues(params, rs, clues, grid);

     desc = snewn(w * h + 1, char);

     for (i = j = 0; i < w*h; i++) {