On Tue, Aug 5, 2014 at 9:35 AM, Robert Haas <robertmh...@gmail.com> wrote:
> On Sat, Aug 2, 2014 at 9:15 AM, Daniele Varrazzo > <daniele.varra...@gmail.com> 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.) > I think you missed one of the regression tests, see attached Thanks, Jeff
regression.diffs
Description: Binary data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
