mirror of
https://git.code.sf.net/p/quake/quakeforge
synced 2025-02-18 18:01:13 +00:00
print the operator for the precedence notice properly
This commit is contained in:
parent
f7d806e8d5
commit
8b24578547
1 changed files with 3 additions and 3 deletions
|
@ -1516,13 +1516,13 @@ binary_expr (int op, expr_t *e1, expr_t *e2)
|
||||||
if ((op == '&' || op == '|' || is_compare (op))
|
if ((op == '&' || op == '|' || is_compare (op))
|
||||||
&& e1->type == ex_uexpr && e1->e.expr.op == '!' && !e1->paren) {
|
&& e1->type == ex_uexpr && e1->e.expr.op == '!' && !e1->paren) {
|
||||||
if (options.traditional) {
|
if (options.traditional) {
|
||||||
notice (e1, "precedence of `!' and `%c' inverted for traditional "
|
notice (e1, "precedence of `!' and `%s' inverted for traditional "
|
||||||
"code", op);
|
"code", get_op_string (op));
|
||||||
e1->e.expr.e1->paren = 1;
|
e1->e.expr.e1->paren = 1;
|
||||||
return unary_expr ('!', binary_expr (op, e1->e.expr.e1, e2));
|
return unary_expr ('!', binary_expr (op, e1->e.expr.e1, e2));
|
||||||
} else if (!is_compare (op)) {
|
} else if (!is_compare (op)) {
|
||||||
warning (e1, "ambiguous logic. Suggest explicit parentheses with "
|
warning (e1, "ambiguous logic. Suggest explicit parentheses with "
|
||||||
"expressions involving ! and %c", op);
|
"expressions involving ! and %s", get_op_string (op));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue