Re: [COMMITTERS] pgsql: Replace most usages of ntoh[ls] and hton[sl]with pg_bswap.h. - Mailing list pgsql-committers

From Michael Paquier
Subject Re: [COMMITTERS] pgsql: Replace most usages of ntoh[ls] and hton[sl]with pg_bswap.h.
Date
Msg-id CAB7nPqS+d4gwwA+RP2q-HDa-1ZAYKe0GRuYNq_8twP5iv0gQKA@mail.gmail.com
Whole thread Raw
In response to [COMMITTERS] pgsql: Replace most usages of ntoh[ls] and hton[sl] with pg_bswap.h.  (Andres Freund <andres@anarazel.de>)
List pgsql-committers
On Mon, Oct 2, 2017 at 8:08 AM, Andres Freund <andres@anarazel.de> wrote:
> Replace most usages of ntoh[ls] and hton[sl] with pg_bswap.h.
>
> All postgres internal usages are replaced, it's just libpq example
> usages that haven't been converted. External users of libpq can't
> generally rely on including postgres internal headers.
>
> Note that this includes replacing open-coded byte swapping of 64bit
> integers (using two 32 bit swaps) with a single 64bit swap.
>
> Where it looked applicable, I have removed netinet/in.h and
> arpa/inet.h usage, which previously provided the relevant
> functionality. It's perfectly possible that I missed other reasons for
> including those, the buildfarm will tell.

Thanks for taking the time to improve that! I was looking for a 64b
equivalent not long ago for pg_rewind... Those changes look good to me
at quick glance.
-- 
Michael


-- 
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

pgsql-committers by date:

Previous
From: Andres Freund
Date:
Subject: [COMMITTERS] pgsql: Replace most usages of ntoh[ls] and hton[sl] with pg_bswap.h.
Next
From: Andres Freund
Date:
Subject: [COMMITTERS] pgsql: Correct include file name in inet_aton fallback.