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>
This commit is contained in:
Khem Raj
2016-12-06 01:22:00 +00:00
committed by Simon Tatham
parent 8dfe5cec31
commit b95812963a
2 changed files with 2 additions and 2 deletions

View File

@ -284,7 +284,7 @@ static int check_nums(game_state *orig, game_state *copy, int only_immutable)
int i, ret = 1; int i, ret = 1;
assert(copy->n == orig->n); assert(copy->n == orig->n);
for (i = 0; i < copy->n; i++) { 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] >= 0);
assert(copy->nums[i] <= copy->n); assert(copy->nums[i] <= copy->n);
if (copy->nums[i] != orig->nums[i]) { if (copy->nums[i] != orig->nums[i]) {

View File

@ -1072,7 +1072,7 @@ static int solve_check_single_sub(game_state *state, int si, int id, int n,
x = i%w; x = i%w;
y = i/w; y = i/w;
if (abs(ox-x) > 1 || abs(oy-y) > 1) { 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); did += solve_set_sflag(state, x, y, S_NOTRACK, what);
} }
} }