From: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | ITAGAKI Takahiro <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] Win32 WEXITSTATUS too |
Date: | 2007-01-22 18:46:25 |
Message-ID: | 20070122184625.GL26006@alvh.no-ip.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Bruce Momjian wrote:
>
> I have applied a modified version of this patch. We now print the
> exception value in hex, and give a URL where the exception can be looked
> up.
Humm, wouldn't it be more appropriate to put the URL in a errhint()
instead?
> + ereport(lev,
> +
> + /*------
> + translator: %s is a noun phrase describing a child process, such as
> + "server process" */
> + (errmsg("%s (PID %d) was terminated by exception %X\nSee http://source.winehq.org/source/include/ntstatus.h for a description\nof the hex value.",
> + procname, pid, WTERMSIG(exitstatus))));
> + #endif
--
Alvaro Herrera http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.
From | Date | Subject | |
---|---|---|---|
Next Message | Merlin Moncure | 2007-01-22 18:49:48 | Re: savepoint improvements |
Previous Message | korryd | 2007-01-22 18:43:29 | Re: 10 weeks to feature freeze (Pending Work) |
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2007-01-22 18:52:11 | Re: [HACKERS] Win32 WEXITSTATUS too |
Previous Message | Bruce Momjian | 2007-01-22 18:33:14 | Re: [HACKERS] Win32 WEXITSTATUS too |