On Sun, Oct 4, 2015 at 2:17 AM, Peter Geoghegan <p...@heroku.com> wrote: > On Tue, Aug 4, 2015 at 12:41 PM, Robert Haas <robertmh...@gmail.com> wrote: >> Some comments: > > I attach a new version of the patch series that incorporates all your > feedback. The patch series is now made cumulative in a way that makes > it easy for someone to independently commit the unsigned integer > comparison optimization for text, and nothing else. The macro that > uses is in a dedicated header now, because I have another patch > (SortSupport for the UUID type) that uses the same optimization for > the same reason. It seems like something that will probably end up > with a third or forth client before too long, so I think the byte swap > macro wrapper belongs in sortsupport.h.
Reviewing 0001, I'm happy to see us add bswap64, but I'm not sure we should put it in c.h, because that's included by absolutely everything. How about putting it in a new #include inside src/port, like src/port/pg_bswap.h? Then pg_crc.h can include that, but other things can, too. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (email@example.com) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers