On Wed, Jan 11, 2017 at 9:42 PM, Peter Eisentraut <peter.eisentr...@2ndquadrant.com> wrote: > There are approximately 11 copies of atooid() and 3 of oid_cmp() or > equivalent, and pending patches are proposing to add more. I propose > these two patches to collect them in central places. > I've verified that the patch covers all the copies of atooid() and oid_cmp() that should be replaced. However, as Tom suggested, I've looked into pg_dump and found that there is a oidcmp() as well. It may have been missed because it has a different name.
I was wondering whether it is worth to replace the following as well: (TransactionId)strtoul(str, NULL, 16) to something like #define atotranid(). -- Thanks & Regards, Kuntal Ghosh EnterpriseDB: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers