Re: Cosmetic change in catalog/index.c - Mailing list pgsql-patches

From Tom Lane
Subject Re: Cosmetic change in catalog/index.c
Date
Msg-id 11415.1045259973@sss.pgh.pa.us
Whole thread Raw
In response to Re: Cosmetic change in catalog/index.c  (greg@turnstep.com)
List pgsql-patches
greg@turnstep.com writes:
> My argument for initial caps is this: the error codes represent complete
> sentences, and look better capitalized. The only exception should be
> when an internal lowercase function name begins the error message, such
> as:
> elog(ERROR, "index_drop: cache lookup failed for index %u",

I would like to migrate *away* from mentioning internal function names
in the error message text at all.  Ideally, information about the
location where the error was reported would be available separately from
the primary message text (I would like elog() to be a macro that can
invoke __FILE__ and __LINE__ behind the scenes).  See past discussions
about revising the error message protocol so that multiple independent
fields can be delivered.

            regards, tom lane

pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: fix regression in .pgpass handling
Next
From: Rod Taylor
Date:
Subject: Object (Domain) locking