On Wed, Aug 6, 2014 at 7:18 PM, Peter Geoghegan <p...@heroku.com> wrote: > On Wed, Aug 6, 2014 at 1:11 PM, Robert Haas <robertmh...@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 -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
