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]
This commit is contained in:
Simon Tatham
2012-01-31 08:32:26 +00:00
parent 070327a440
commit f3168895c8

View File

@ -1335,6 +1335,8 @@ static int new_clues(game_params *params, random_state *rs,
break; /* got it */
}
debug(("%d %dx%d loops before finished puzzle.\n", ngen, w, h));
return ngen;
}
@ -1343,14 +1345,12 @@ static char *new_game_desc(game_params *params, random_state *rs,
{
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++) {