From: Christoph Mallon <christoph.mallon@gmx.de>
Date: Fri, 14 Dec 2007 16:54:15 +0000 (+0000)
Subject: Make warning about statement without effect less aggressive for && and ||.
X-Git-Url: http://nsz.repo.hu/git/?a=commitdiff_plain;h=4c07f7bd844d1ce4fc3f276ed19d85ec9e0843a5;p=cparser

Make warning about statement without effect less aggressive for && and ||.

[r18743]
---

diff --git a/parser.c b/parser.c
index 53707c3..c14cb0e 100644
--- a/parser.c
+++ b/parser.c
@@ -4487,8 +4487,6 @@ static bool expression_has_effect(const expression_t *const expr)
 		case EXPR_BINARY_BITWISE_AND:        return false;
 		case EXPR_BINARY_BITWISE_OR:         return false;
 		case EXPR_BINARY_BITWISE_XOR:        return false;
-		case EXPR_BINARY_LOGICAL_AND:        return false;
-		case EXPR_BINARY_LOGICAL_OR:         return false;
 		case EXPR_BINARY_SHIFTLEFT:          return false;
 		case EXPR_BINARY_SHIFTRIGHT:         return false;
 		case EXPR_BINARY_ASSIGN:             return true;
@@ -4502,6 +4500,8 @@ static bool expression_has_effect(const expression_t *const expr)
 		case EXPR_BINARY_BITWISE_AND_ASSIGN: return true;
 		case EXPR_BINARY_BITWISE_XOR_ASSIGN: return true;
 		case EXPR_BINARY_BITWISE_OR_ASSIGN:  return true;
+		case EXPR_BINARY_LOGICAL_AND:
+		case EXPR_BINARY_LOGICAL_OR:
 		case EXPR_BINARY_COMMA:
 			return expression_has_effect(expr->binary.right);