Re: phrase search TS_phrase_execute code readability patch - Mailing list pgsql-hackers

From Robert Haas
Subject Re: phrase search TS_phrase_execute code readability patch
Date
Msg-id CA+TgmoYUisVETnoVPL_ynE_d6WTKfui1jtqN_mm51B_EvAaeCw@mail.gmail.com
Whole thread Raw
In response to phrase search TS_phrase_execute code readability patch  ("David G. Johnston" <david.g.johnston@gmail.com>)
List pgsql-hackers
On Tue, Aug 9, 2016 at 3:35 PM, David G. Johnston
<david.g.johnston@gmail.com> wrote:
> I don't follow why LposStart is needed so I removed it...

That doesn't seem very reasonable.

> Not compiled or in any way tested...

Please do not bother submitting patches that you aren't prepared to
compile and test.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: No longer possible to query catalogs for index capabilities?
Next
From: Kisung Kim
Date:
Subject: Re: Btree Index on PostgreSQL and Wiredtiger (MongoDB3.2)