Re: Replace open mode with PG_BINARY_R/W/A macros - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: Replace open mode with PG_BINARY_R/W/A macros
Date
Msg-id 4e7545a9-7cba-6be2-0a6e-72a0f531184a@enterprisedb.com
Whole thread Raw
In response to Re: Replace open mode with PG_BINARY_R/W/A macros  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Replace open mode with PG_BINARY_R/W/A macros  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
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.




pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: BufferAlloc: don't take two simultaneous locks
Next
From: Peter Eisentraut
Date:
Subject: Re: [RFC] building postgres with meson -v8