Duncan Smith <Duncan@xrtc.net> writes:
> I noticed today that the network encoding of the 'inet' type uses values
> derived from the system header file <sys/socket.h> to tag whether the
> address is IPv4 or IPv6.
> In particular, <src/include/utils/inet.h> defines PGSQL_AF_INET to be
> AF_INET, which is 2 on my system. Wouldn't it be better to use a more
> constant constant for this purpose?
Yeah, in a green field that would have been the thing to do, but at
this point it's years too late to change it. If there are any platforms
where AF_INET is different from 2, changing those constants would break
both on-disk storage and binary-format dumps for them.
regards, tom lane