From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Magnus Hagander <magnus(at)hagander(dot)net> |
Cc: | Takayuki Tsunakawa <tsunakawa(dot)takay(at)jp(dot)fujitsu(dot)com>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Alvaro Herrera <alvherre(at)commandprompt(dot)com>, ITAGAKI Takahiro <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp> |
Subject: | Re: [HACKERS] Win32 WEXITSTATUS too |
Date: | 2007-01-23 14:29:19 |
Message-ID: | 200701231429.l0NETJl07063@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Magnus Hagander wrote:
> On Tue, Jan 23, 2007 at 06:50:06PM +0900, Takayuki Tsunakawa wrote:
> > From: "Magnus Hagander" <magnus(at)hagander(dot)net>
> > > Are you entirely sure that ntstatus.h is where to look? Because per
> > > whatever docs I've found, that contains "device driver errors" and
> > *not*
> > > exception codes.
> >
> > Yes, what you are pointing out is correct. winbase.h and winnt.h
> > should be consulted instead of ntstatus.h. See the the section
> > "Return Value" in the following page:
> >
> > http://msdn2.microsoft.com/ru-ru/library/ms679356.aspx
> >
> > Furthermore, the message is meaningless for users because they can do
> > nothing with the information. So, I think the message should say
> > something like
> >
> > child process was terminated by exception %X
> > This seems to be a bug of PostgreSQL.
> > Please report this message with the details of the phynomenon to
> > PostgreSQL developers.
> >
> >
> > What do you think?
>
> I think that's incorrect information to the user :-(
> If the child terminates with exit(1), we will then say "child process
> was terminated by exception 1. This seems to be a bug", which is clearly
> not true.
>
> Unless you know a sure way of determining if the exitcode is a normal
> exitcode or an exception code.
Current CVS believes values >= 0x100 are non-exit() terminations.
--
Bruce Momjian bruce(at)momjian(dot)us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2007-01-23 14:31:40 | Re: msvc failure in largeobject regression test |
Previous Message | Stefan Kaltenbrunner | 2007-01-23 14:21:13 | Re: msvc failure in largeobject regression test |
From | Date | Subject | |
---|---|---|---|
Next Message | Magnus Hagander | 2007-01-23 15:07:43 | Re: [HACKERS] Win32 WEXITSTATUS too |
Previous Message | Magnus Hagander | 2007-01-23 14:02:01 | vc++ regression tests |