Commit930930c

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 parent036f7d3 commit930930cCopy full SHA for 930930c

File tree

-1

lines changed

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -3154,7 +3154,7 @@ operator_precedence_group(Node node, const char *nodename)`
`3154`	`3154`	`*nodename=strVal(linitial(aexpr->name));`
`3155`	`3155`	`/* Ignore if op was always higher priority than IS-tests */`
`3156`	`3156`	`if (strcmp(*nodename,"+")==0\|\|`
`3157`		`-strcmp(*nodename,"-"))`
	`3157`	`+strcmp(*nodename,"-")==0)`
`3158`	`3158`	`group=0;`
`3159`	`3159`	`else`
`3160`	`3160`	`group=PREC_GROUP_PREFIX_OP;`

Comments

(0)