Commit4cae471

committed

Fix backwards test in operator_precedence_warning logic.

Warnings about unary minus might have been wrong. It's a bitsurprising that nobody noticed yet ... probably the precedence-warningfeature hasn't really been used much in the field.Rikard FalkebornDiscussion:https://postgr.es/m/CADRDgG6fzA8A2oeygUw4=o7ywo4kvz26NxCSgpq22nMD73Bx4Q@mail.gmail.com

1 parent765525c commit4cae471Copy full SHA for 4cae471

File tree

-1

lines changed

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -3236,7 +3236,7 @@ operator_precedence_group(Node node, const char *nodename)`
`3236`	`3236`	`*nodename=strVal(linitial(aexpr->name));`
`3237`	`3237`	`/* Ignore if op was always higher priority than IS-tests */`
`3238`	`3238`	`if (strcmp(*nodename,"+")==0\|\|`
`3239`		`-strcmp(*nodename,"-"))`
	`3239`	`+strcmp(*nodename,"-")==0)`
`3240`	`3240`	`group=0;`
`3241`	`3241`	`else`
`3242`	`3242`	`group=PREC_GROUP_PREFIX_OP;`

Comments

(0)