Le 28/02/2018 à 23:32, Junio C Hamano a écrit :
> Eric Sunshine <sunsh...@sunshineco.com> writes:
>
>> On Wed, Feb 28, 2018 at 5:17 PM, Alban Gruin <alban.gr...@gmail.com> wrote:
>>> Yes, but I can split the line like that:
>>>
>>> % cat >baz.go<<\EOF
>>> package baz
>>> func baz(arg1 int64,
>>> arg2 int64) {
>>> }
>>> EOF
>>> % go build baz.go
>>>
>>> This complies to the standard formatting (at least, gofmt doesn't change
>>> it), but making the regex strict about the brace would cause it to
>>> ignore those funcs, although I don't know how common they are.
>>
>> Makes sense. Thanks for the clarifying example. I wouldn't be at all
>> surprised it such formatting exists in the wild, so keeping the regex
>> as-is seems a good idea.
>
> Does input like that appear in the tests the patch adds? If not, it
> probably is a good idea to have it somewhere in the commit (even if
> there is no test addition, having it in the log message to explain
> why the regex is done like so would be a good idea).
>
> Thanks.
>
Yes, it's in a file called "golang-long-func". I'll send another patch
later to fix the typo in the commit message and the indentation in the
tests. I'll clarify the regex in the message, too.
