aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKhem Raj <raj.khem@gmail.com>2016-12-06 01:22:00 +0000
committerSimon Tatham <anakin@pobox.com>2016-12-06 21:40:24 +0000
commitb95812963a8d187526145f5e1a9693f10eee2b65 (patch)
tree50369a89b2cefe103047788ae3364f3be7b751dd
parent8dfe5cec31e784e4ece2955ecc8cc35ee7e8fbb3 (diff)
downloadpuzzles-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.c2
-rw-r--r--tracks.c2
2 files changed, 2 insertions, 2 deletions
diff --git a/signpost.c b/signpost.c
index aa2e13a..2e2dff2 100644
--- a/signpost.c
+++ b/signpost.c
@@ -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]) {
diff --git a/tracks.c b/tracks.c
index 5b27350..ca44ce1 100644
--- a/tracks.c
+++ b/tracks.c
@@ -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);
}
}