On Wed, Jan 12, 2005 at 09:23:26AM +1100, Brendan Jurd wrote:
> This patch to src/backend/executor/nodeFunctionscan.c is intended to 
> make life a little easier for people using row-returning functions, by 
> increasing the level of detail in the error messages thrown when 
> tupledesc_match fails.

You should get rid of the returns, because ereport(ERROR) will never
return control to the function and they are thus dead code.  And make
the function return void rather than bool.

Also follow the style: use "if (foo)" rather than "if( foo )".  And
message style stipulates that the errdetail() message should start with
a capital (upper case?) letter.

Alvaro Herrera (<[EMAIL PROTECTED]>)
"Pido que me den el Nobel por razones humanitarias" (Nicanor Parra)

---------------------------(end of broadcast)---------------------------
TIP 6: Have you searched our list archives?


Reply via email to