diff options
| author | Simon Tatham <anakin@pobox.com> | 2018-11-07 19:16:00 +0000 |
|---|---|---|
| committer | Simon Tatham <anakin@pobox.com> | 2018-11-07 19:17:47 +0000 |
| commit | b732fda2cf611b6f67c68e3a26077897afe41741 (patch) | |
| tree | e17dd06e409b85c3299b7cfee1bbbbf68fb14b63 /undead.c | |
| parent | 3189fb484dcfe4efe42b2b30e529389db9dd0d00 (diff) | |
| download | puzzles-b732fda2cf611b6f67c68e3a26077897afe41741.zip puzzles-b732fda2cf611b6f67c68e3a26077897afe41741.tar.gz puzzles-b732fda2cf611b6f67c68e3a26077897afe41741.tar.bz2 puzzles-b732fda2cf611b6f67c68e3a26077897afe41741.tar.xz | |
Undead: remove an unused structure field.
I noticed that state->common, which is shared between all the game
states in an undo chain, has a 'solved' flag in it. That's not right -
solvedness as a property of a particular state on the chain belongs in
the game_state itself, and having-at-one-point-been-solved-ness as a
persistent property of the whole chain belongs in the game_ui.
Fortunately, the game isn't actually doing it wrong:
state->common->solved is set once and then never read, so it must have
been left in from early abandoned code. Now removed.
Diffstat (limited to 'undead.c')
| -rw-r--r-- | undead.c | 2 |
1 files changed, 0 insertions, 2 deletions
@@ -225,7 +225,6 @@ struct game_common { int *grid; int *xinfo; int *fixed; - int solved; }; struct game_state { @@ -260,7 +259,6 @@ static game_state *new_state(const game_params *params) { state->common->grid = snewn(state->common->wh, int); state->common->xinfo = snewn(state->common->wh, int); state->common->fixed = NULL; - state->common->solved = FALSE; state->common->num_paths = state->common->params.w + state->common->params.h; |