| 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: | Whole Thread | Raw Message | 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 |