diff options
| author | Khem Raj <raj.khem@gmail.com> | 2016-12-06 01:22:00 +0000 |
|---|---|---|
| committer | Simon Tatham <anakin@pobox.com> | 2016-12-06 21:40:24 +0000 |
| commit | b95812963a8d187526145f5e1a9693f10eee2b65 (patch) | |
| tree | 50369a89b2cefe103047788ae3364f3be7b751dd | |
| parent | 8dfe5cec31e784e4ece2955ecc8cc35ee7e8fbb3 (diff) | |
| download | puzzles-b95812963a8d187526145f5e1a9693f10eee2b65.zip puzzles-b95812963a8d187526145f5e1a9693f10eee2b65.tar.gz puzzles-b95812963a8d187526145f5e1a9693f10eee2b65.tar.bz2 puzzles-b95812963a8d187526145f5e1a9693f10eee2b65.tar.xz | |
Clarify conditions to avoid compiler errors
Fix errors pointed out by clang
error: logical not is only applied to the left hand side of this bitwise operator [-Werror,-Wlogical-not-parentheses]
| if (only_immutable && !copy->flags[i] & FLAG_IMMUTABLE) continue;
| ^
Signed-off-by: Khem Raj <raj.khem@gmail.com>
| -rw-r--r-- | signpost.c | 2 | ||||
| -rw-r--r-- | tracks.c | 2 |
2 files changed, 2 insertions, 2 deletions
@@ -284,7 +284,7 @@ static int check_nums(game_state *orig, game_state *copy, int only_immutable) int i, ret = 1; assert(copy->n == orig->n); for (i = 0; i < copy->n; i++) { - if (only_immutable && !copy->flags[i] & FLAG_IMMUTABLE) continue; + if (only_immutable && !(copy->flags[i] & FLAG_IMMUTABLE)) continue; assert(copy->nums[i] >= 0); assert(copy->nums[i] <= copy->n); if (copy->nums[i] != orig->nums[i]) { @@ -1072,7 +1072,7 @@ static int solve_check_single_sub(game_state *state, int si, int id, int n, x = i%w; y = i/w; if (abs(ox-x) > 1 || abs(oy-y) > 1) { - if (!state->sflags[i] & S_TRACK) + if (!(state->sflags[i] & S_TRACK)) did += solve_set_sflag(state, x, y, S_NOTRACK, what); } } |