Sender: a.joubert@albourne.com
Message-ID: <37EA3FB8.95336FAE@albourne.com>
Date: Thu, 23 Sep 1999 17:56:56 +0300
From: Adriaan Joubert <a.joubert@albourne.com>
Organization: APL Financial Services (Overseas) Ltd
MIME-Version: 1.0
To: Tom Lane <tgl@sss.pgh.pa.us>
CC: Postgresql <pgsql-hackers@postgreSQL.org>,
	Thomas Lockhart <lockhart@alumni.caltech.edu>
Subject: Re: [HACKERS] Operator definitions
References: <14154.938097658@sss.pgh.pa.us>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit

Tom Lane wrote:
> 
> Adriaan Joubert <a.joubert@albourne.com> writes:
> > 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.
> 
> If you search for, eg, '%', you will find there are several production
> lists that call out all the operators; your patch only caught one of them.
> 

Well, as I said, I don't really understand what is going on in that file
and I added the minimum to make my stuff work. Thomas said he was going
to have a look at it, so I think I'll rely on him fixing it, before I
break anything else ;-). 

I've already noticed that I will need | as an operator for the SQL bit
types, so I may have to hack it a bit more. I just hate changing things
blindly I don't really understand. Thanks for pointing it out though!

Adriaan