On 20.04.22 22:29, Tom Lane wrote:
> PG_BINARY is useful for open(). It's the PG_BINARY_R/W/A macros for
> fopen() that are redundant per POSIX. Possibly someone generalized
> inappropriately; or maybe long ago we supported some platform that
> rejected the "b" option?
I think the latter was the case. I doubt it's still a problem.
I see some of the new code in pg_basebackup uses "wb" directly. It
would probably be good to fix that to be consistent one way or the
other. I vote for getting rid of the macros.