On Sat, Aug 2, 2014 at 9:15 AM, Daniele Varrazzo <[email protected]> wrote: > I'd definitely replace /arg/argument/. Furthermore I'd avoid the form > "argument 1: something is wrong": the string is likely to end up in > sentences with other colons so I'd rather use "something is wrong at > argument 1". > > Is the patch attached better?
Looks OK to me. I thought someone else might comment, but since no one has, committed. (As a note for the future, you forgot to update the regression test outputs; I took care of that before committing.) -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
