On Sat, Aug 2, 2014 at 9:15 AM, Daniele Varrazzo <daniele.varrazzo@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
