diff options
| author | Simon Tatham <anakin@pobox.com> | 2023-06-16 19:03:56 +0100 |
|---|---|---|
| committer | Simon Tatham <anakin@pobox.com> | 2023-06-16 19:04:50 +0100 |
| commit | c82537b4574d45aa16e50b7f8dc1f075cfdb69f9 (patch) | |
| tree | f830b661a3e20d4d1ec996d4e1a3880615261da1 /mosaic.c | |
| parent | de13ca2874e673b426efcf04f734ae6625635396 (diff) | |
| download | puzzles-c82537b4574d45aa16e50b7f8dc1f075cfdb69f9.zip puzzles-c82537b4574d45aa16e50b7f8dc1f075cfdb69f9.tar.gz puzzles-c82537b4574d45aa16e50b7f8dc1f075cfdb69f9.tar.bz2 puzzles-c82537b4574d45aa16e50b7f8dc1f075cfdb69f9.tar.xz | |
Fix some unused-variable warnings.
A test-build with a modern clang points out a number of 'set but not
used' variables, which clang seems to have got better at recently.
In cases where there's conditioned-out or commented-out code using the
variable, I've left it in and added a warning-suppressing cast to
void. Otherwise I've just deleted the variables.
Diffstat (limited to 'mosaic.c')
| -rw-r--r-- | mosaic.c | 6 |
1 files changed, 2 insertions, 4 deletions
@@ -409,7 +409,7 @@ static void count_clues_around(const game_params *params, static void mark_around(const game_params *params, struct solution_cell *sol, int x, int y, int mark) { - int i, j, marked = 0; + int i, j; struct solution_cell *curr; for (i = -1; i < 2; i++) { @@ -418,7 +418,6 @@ static void mark_around(const game_params *params, if (curr) { if (curr->cell == STATE_UNMARKED) { curr->cell = mark; - marked++; } } } @@ -585,7 +584,7 @@ static bool solve_game_actual(const game_params *params, int board_size = params->height * params->width; struct solution_cell *sol = snewn(board_size, struct solution_cell); bool made_progress = true, error = false; - int solved = 0, iter = 0, curr = 0; + int solved = 0, curr = 0; memset(sol, 0, params->height * params->width * sizeof(*sol)); solved = 0; @@ -607,7 +606,6 @@ static bool solve_game_actual(const game_params *params, solved += curr; } } - iter++; } if (sol_return) { *sol_return = sol; |