On May 29, 1:04 pm, t...@sss.pgh.pa.us (Tom Lane) wrote: > Kevin Field <kevinjamesfi...@gmail.com> writes: > > default: > > elog(ERROR, "unrecognized raise option: %d", > > opt->opt_type); > > Should this be changed to: > > default: > > ereport(ERROR, (errmsg_internal("unrecognized > > raise option: %d", > > opt->opt_type))); > > No, we generally don't bother with that. The above two are exactly > equivalent and the first is easier to write, so why complicate the code? > ereport is needed if you want to specify a SQLSTATE, provide a > translatable error message, etc, but for internal shouldn't-happen cases > we customarily just use elog.
Ah, I had missed that. I understand. The function's comment's still out of date though, I think, since it uses ereport at the end. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers