pgsql: Add quotes to message

Started by Nonamealmost 17 years ago3 messages
#1Noname
petere@postgresql.org

Log Message:
-----------
Add quotes to message

Modified Files:
--------------
pgsql/src/pl/plpgsql/src:
pl_exec.c (r1.234 -> r1.235)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/pl/plpgsql/src/pl_exec.c?r1=1.234&r2=1.235)

#2Gregory Stark
stark@enterprisedb.com
In reply to: Noname (#1)
Re: pgsql: Add quotes to message

petere@postgresql.org (Peter Eisentraut) writes:

Log Message:
-----------
Add quotes to message

errdetail("Returned type %s does not match expected type "
! "%s in column \"%s\".",
OidIsValid(returned->attrs[i]->atttypid) ?
format_type_be(returned->attrs[i]->atttypid) :
_(dropped_column_type),

I'm surprised there isn't a regression test for this case.

--
Gregory Stark
EnterpriseDB http://www.enterprisedb.com
Ask me about EnterpriseDB's 24x7 Postgres support!

#3Peter Eisentraut
peter_e@gmx.net
In reply to: Gregory Stark (#2)
Re: pgsql: Add quotes to message

Gregory Stark wrote:

petere@postgresql.org (Peter Eisentraut) writes:

Log Message:
-----------
Add quotes to message

errdetail("Returned type %s does not match expected type "
! "%s in column \"%s\".",
OidIsValid(returned->attrs[i]->atttypid) ?
format_type_be(returned->attrs[i]->atttypid) :
_(dropped_column_type),

I'm surprised there isn't a regression test for this case.

Me too actually. ;-) In general, the plpgsql test doesn't appear to
cover the error paths very well at all. I changed the wording of about
50 error messages the other day and only two or three of those appeared
in a regression test case.

(The test coverage framework doesn't cover the PLs yet, so I don't have
anything more precise to say on that.)