aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2019-04-10 19:02:21 -0400
committerTom Lane <tgl@sss.pgh.pa.us>2019-04-10 19:02:58 -0400
commit99bbff5c15797de3d958f303e80646d30e0c8fa4 (patch)
tree6b7666211e38f9c9c1d485c73fc5a0e7b41b1f1f
parent7657072e0ccc01955fddfce22ffcc4d82f9e9e45 (diff)
downloadpostgresql-99bbff5c15797de3d958f303e80646d30e0c8fa4.tar.gz
postgresql-99bbff5c15797de3d958f303e80646d30e0c8fa4.zip
Fix backwards test in operator_precedence_warning logic.
Warnings about unary minus might have been wrong. It's a bit surprising that nobody noticed yet ... probably the precedence-warning feature hasn't really been used much in the field. Rikard Falkeborn Discussion: https://postgr.es/m/CADRDgG6fzA8A2oeygUw4=o7ywo4kvz26NxCSgpq22nMD73Bx4Q@mail.gmail.com
-rw-r--r--src/backend/parser/parse_expr.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/parser/parse_expr.c b/src/backend/parser/parse_expr.c
index 6d8cb07766b..197c432dd41 100644
--- a/src/backend/parser/parse_expr.c
+++ b/src/backend/parser/parse_expr.c
@@ -3145,7 +3145,7 @@ operator_precedence_group(Node *node, const char **nodename)
*nodename = strVal(linitial(aexpr->name));
/* Ignore if op was always higher priority than IS-tests */
if (strcmp(*nodename, "+") == 0 ||
- strcmp(*nodename, "-"))
+ strcmp(*nodename, "-") == 0)
group = 0;
else
group = PREC_GROUP_PREFIX_OP;