From 2efc77d2fde7e53604f8490b57f18a36eec5c4fb Mon Sep 17 00:00:00 2001 From: Simon Tatham Date: Wed, 4 May 2011 18:22:14 +0000 Subject: Fix warnings generated by gcc 4.6.0 about variables set but not thereafter read. Most of these changes are just removal of pointless stuff or trivial reorganisations; one change is actually substantive, and fixes a bug in Keen's clue selection (the variable 'bad' was unreferenced not because I shouldn't have set it, but because I _should_ have referenced it!). [originally from svn r9164] --- lightup.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'lightup.c') diff --git a/lightup.c b/lightup.c index 5b97b2b..5d27fde 100644 --- a/lightup.c +++ b/lightup.c @@ -1402,6 +1402,7 @@ static int strip_unused_nums(game_state *state) } } } + debug(("Stripped %d unused numbers.\n", n)); return n; } @@ -1475,7 +1476,7 @@ static char *new_game_desc(game_params *params, random_state *rs, char **aux, int interactive) { game_state *news = new_state(params), *copys; - int nsol, i, j, run, x, y, wh = params->w*params->h, num; + int i, j, run, x, y, wh = params->w*params->h, num; char *ret, *p; int *numindices; @@ -1499,8 +1500,7 @@ static char *new_game_desc(game_params *params, random_state *rs, /* Take a copy, remove numbers we didn't use and check there's * still a unique solution; if so, use the copy subsequently. */ copys = dup_game(news); - nsol = strip_unused_nums(copys); - debug(("Stripped %d unused numbers.\n", nsol)); + strip_unused_nums(copys); if (!puzzle_is_good(copys, params->difficulty)) { debug(("Stripped grid is not good, reverting.\n")); free_game(copys); -- cgit v1.1