From 72f2e9043ef509992dfa61791638db87bf0ae37b Mon Sep 17 00:00:00 2001 From: Khem Raj Date: Tue, 6 Dec 2016 01:20:00 +0000 Subject: [PATCH] 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 --- Upstream-Status: Submitted signpost.c | 2 +- tracks.c | 2 +- 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); } } -- 1.9.1