Commit99bbff5

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 parent7657072 commit99bbff5Copy full SHA for 99bbff5

File tree

-1

lines changed

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -3145,7 +3145,7 @@ operator_precedence_group(Node node, const char *nodename)`
`3145`	`3145`	`*nodename=strVal(linitial(aexpr->name));`
`3146`	`3146`	`/* Ignore if op was always higher priority than IS-tests */`
`3147`	`3147`	`if (strcmp(*nodename,"+")==0\|\|`
`3148`		`-strcmp(*nodename,"-"))`
	`3148`	`+strcmp(*nodename,"-")==0)`
`3149`	`3149`	`group=0;`
`3150`	`3150`	`else`
`3151`	`3151`	`group=PREC_GROUP_PREFIX_OP;`

Comments

(0)