From: Tom Lane <tgl@sss.pgh.pa.us>
To: Pierre Forstmann <pierre.forstmann@gmail.com>
cc: pierre@senellart.com, pgsql-bugs@lists.postgresql.org
Subject: Re: [patch] BUG #19558: User-defined prefix operators "|" and "->" no
 longer parse in 19beta2 (SQL/PGQ grammar change)
In-reply-to: <1b69cad1-19b4-4334-bb6e-a343151e8b78@gmail.com>
References: <19558-ad1fca59a3a471a0@postgresql.org>
 <02aea982-c656-462d-a412-1fb2e388f754@gmail.com>
 <11151.1784389359@sss.pgh.pa.us>
 <1b69cad1-19b4-4334-bb6e-a343151e8b78@gmail.com>
Comments: In-reply-to Pierre Forstmann <pierre.forstmann@gmail.com>
	message dated "Sat, 18 Jul 2026 18:01:48 +0200"
MIME-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
Content-ID: <125920.1784394211.1@sss.pgh.pa.us>
Content-Transfer-Encoding: quoted-printable
Date: Sat, 18 Jul 2026 13:03:31 -0400
Message-ID: <125921.1784394211@sss.pgh.pa.us>
Archived-At: 
 <https://www.postgresql.org/message-id/125921.1784394211%40sss.pgh.pa.us>
Precedence: bulk

Pierre Forstmann <pierre.forstmann@gmail.com> writes:
> Le 18/07/2026 =C3=A0 17:42, Tom Lane a =C3=A9crit=C2=A0:
>> 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