Re: [patch] BUG #19558: User-defined prefix operators "|" and "->" no longer parse in 19beta2 (SQL/PGQ grammar change)

Pierre Forstmann <pierre(dot)forstmann(at)gmail(dot)com> writes:
> Le 18/07/2026 à 17:42, Tom Lane a écrit :
>> Why are you promoting these to UMINUS binding precedence?
>> AFAICS they would have had Op precedence before v19
>> (and still do, as binary operators).

> No good reason: I have fixed my mistake in patch v002.

OK. You don't need the explicit %prec specs, because these operators
already have the same precedence as Op:

%left Op OPERATOR RIGHT_ARROW '|' /* multi-character ops and user-defined operators */

and a bison rule takes the precedence of its rightmost terminal
symbol by default. More importantly, you missed adding the
same productions to b_expr, so that there would have been contexts
where these symbols still didn't work like they used to.

I fixed both those things and pushed it. Thanks for the
report and the patch!

regards, tom lane