Alvaro Herrera wrote:
> 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

Oops, forgot to mention that detail.  We are using log_error() in one
case, and ereport() in another.  Let me do the hint in the report case,
but I have to leave the log_error case alone because it takes only three
arguments.

-- 
  Bruce Momjian   [EMAIL PROTECTED]
  EnterpriseDB    http://www.enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

---------------------------(end of broadcast)---------------------------
TIP 9: In versions below 8.0, the planner will ignore your desire to
       choose an index scan if your joining column's datatypes do not
       match

Reply via email to