From: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi> |
---|---|
To: | Tristan Partin <tristan(at)neon(dot)tech>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Remove WIN32 conditional compilation from win32common.c |
Date: | 2024-02-12 09:58:24 |
Message-ID: | 87d1ba81-0fe8-4ef6-912b-afdc751beae2@iki.fi |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 05/12/2023 20:37, Tristan Partin wrote:
> The file is only referenced in Meson and MSVC scripts from what I can
> tell, and the Meson reference is protected by a Windows check.
There are a bunch of files like win32common.c:
$ ls src/port/win32*.c
src/port/win32common.c
src/port/win32dlopen.c
src/port/win32env.c
src/port/win32error.c
src/port/win32fdatasync.c
src/port/win32fseek.c
src/port/win32gai_strerror.c
src/port/win32getrusage.c
src/port/win32gettimeofday.c
src/port/win32link.c
src/port/win32ntdll.c
src/port/win32pread.c
src/port/win32pwrite.c
src/port/win32security.c
src/port/win32setlocale.c
src/port/win32stat.c
Of these, win32stat.c and win32fseek.c also contain "#ifdef WIN32", but
others don't. So I concur that the most common pattern in these files is
to not use #ifdef WIN32, and +1 for making them consistent.
I removed those from win32stat.c and win32fseek.c, too, and committed.
Thanks!
--
Heikki Linnakangas
Neon (https://neon.tech)
From | Date | Subject | |
---|---|---|---|
Next Message | Bertrand Drouvot | 2024-02-12 10:03:20 | Re: Synchronizing slots from primary to standby |
Previous Message | Peter Eisentraut | 2024-02-12 09:55:09 | Re: SQL:2011 application time |