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