Received: from localhost (IDENT:root@hectic-1.jpl.nasa.gov [128.149.68.203]) by hub.org (8.9.3/8.9.3) with ESMTP id JAA45216 for ; Thu, 23 Sep 1999 09:34:11 -0400 (EDT) (envelope-from lockhart@alumni.caltech.edu) Received: from alumni.caltech.edu (lockhart@localhost [127.0.0.1]) by localhost (8.8.7/8.8.7) with ESMTP id NAA21288; Thu, 23 Sep 1999 13:32:50 GMT Sender: lockhart@hub.org Message-ID: <37EA2C02.C0759C88@alumni.caltech.edu> Date: Thu, 23 Sep 1999 13:32:50 +0000 From: Thomas Lockhart X-Mailer: Mozilla 4.6 [en] (X11; I; Linux 2.0.36 i686) X-Accept-Language: en MIME-Version: 1.0 To: Adriaan Joubert , Bruce Momjian , Lamar Owen CC: Tom Lane , Postgresql Subject: Re: [HACKERS] Operator definitions References: <2198.938033780@sss.pgh.pa.us> <37E9DA85.F0210261@albourne.com> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit > > It looks to me like '^' and '|' have been left out of the alternatives > > for MathOp in src/backend/parser/gram.y. It could be they were > > deliberately omitted, but I bet it's just an oversight. > OK, here is a patch to allow both ^ and | as operators, both in operator > definitions and expressions. It seems to work for me. Unfortunately the > regression tests do not tell me an awful lot, as several of them fail on > the Alpha anyway. As I don;t really know what I'm doing, I'd appreciate > it if somebody else could check the patch out and let me know whether it > is ok. It's fine as far as it goes, but istm that there are other cases missing from gram.y also :( Bruce, how did this stuff get into the stable release? Looks like we are going to need a v6.5.3 Real Soon Now. And packagers, we should plan on having a patch for v6.5.2. I'll try coming up with one in the next couple of days; I've tested on my own gram.y but it has too many other changes to be used as-is. - Thomas -- Thomas Lockhart lockhart@alumni.caltech.edu South Pasadena, California