On Wed, Aug 6, 2014 at 7:18 PM, Peter Geoghegan <pg@heroku.com> wrote:
> On Wed, Aug 6, 2014 at 1:11 PM, Robert Haas <robertmhaas@gmail.com> wrote:
>> I've committed the patch I posted yesterday. I did not see a good
>> reason to meld that together in a single commit with the first of the
>> patches you posted; I'll leave you to revise that patch to conform
>> with this approach.
>
> Okay. Attached is the same patch set, rebased on top on your commit
> with appropriate amendments.
Two things:
+ * result. Also, since strxfrm()/strcoll() require
NULL-terminated inputs,
In my original patch, I wrote NUL, as in the NUL character. You've
changed it to NULL, but the original was correct. NULL is a pointer
value that is not relevant here; the character with value 0 is NUL.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company