pgsql: Minor improvement: avoid assuming that GetLastError value cannot

Started by Tom Laneabout 17 years ago4 messagescomitters
Jump to latest
#1Tom Lane
tgl@sss.pgh.pa.us

Log Message:
-----------
Minor improvement: avoid assuming that GetLastError value cannot be
affected by CloseHandle() or Sleep().

Modified Files:
--------------
pgsql/src/backend/port:
win32_shmem.c (r1.9 -> r1.10)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/port/win32_shmem.c?r1=1.9&r2=1.10)

#2Magnus Hagander
magnus@hagander.net
In reply to: Tom Lane (#1)
Re: pgsql: Minor improvement: avoid assuming that GetLastError value cannot

Any reason you didn't backport this to the other versions that got the
SetLastError() fix?

//Magnus

Tom Lane wrote:

Show quoted text

Log Message:
-----------
Minor improvement: avoid assuming that GetLastError value cannot be
affected by CloseHandle() or Sleep().

Modified Files:
--------------
pgsql/src/backend/port:
win32_shmem.c (r1.9 -> r1.10)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/port/win32_shmem.c?r1=1.9&r2=1.10)

#3Tom Lane
tgl@sss.pgh.pa.us
In reply to: Magnus Hagander (#2)
Re: pgsql: Minor improvement: avoid assuming that GetLastError value cannot

Magnus Hagander <magnus@hagander.net> writes:

Any reason you didn't backport this to the other versions that got the
SetLastError() fix?

AFAIK it's just a cosmetic improvement. If you think there's a
significant risk then back-port away...

regards, tom lane

#4Magnus Hagander
magnus@hagander.net
In reply to: Tom Lane (#3)
Re: pgsql: Minor improvement: avoid assuming that GetLastError value cannot

Tom Lane wrote:

Magnus Hagander <magnus@hagander.net> writes:

Any reason you didn't backport this to the other versions that got the
SetLastError() fix?

AFAIK it's just a cosmetic improvement. If you think there's a
significant risk then back-port away...

Not really, but it'd be easier if we need to backpatch more stuff later
if the source files are the same at least :-)

//Magnus