From f3168895c89e706c53c881f53ba3dfc0fc1150ec Mon Sep 17 00:00:00 2001 From: Simon Tatham Date: Tue, 31 Jan 2012 08:32:26 +0000 Subject: [PATCH] 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] --- pearl.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/pearl.c b/pearl.c index 02a43db..248d64e 100644 --- a/pearl.c +++ b/pearl.c @@ -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++) {