aboutsummaryrefslogtreecommitdiff
path: root/undead.c
diff options
context:
space:
mode:
authorSimon Tatham <anakin@pobox.com>2018-11-07 19:16:00 +0000
committerSimon Tatham <anakin@pobox.com>2018-11-07 19:17:47 +0000
commitb732fda2cf611b6f67c68e3a26077897afe41741 (patch)
treee17dd06e409b85c3299b7cfee1bbbbf68fb14b63 /undead.c
parent3189fb484dcfe4efe42b2b30e529389db9dd0d00 (diff)
downloadpuzzles-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.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/undead.c b/undead.c
index 03b014f..6a6b338 100644
--- a/undead.c
+++ b/undead.c
@@ -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;