>> diff --git a/userdiff.c b/userdiff.c
>> @@ -38,6 +38,15 @@ IPATTERN("fortran",
>> +PATTERNS("golang",
>> + /* Functions */
>> + "^[ \t]*(func[ \t]*.*(\\{[ \t]*)?)\n"
>
> Why is the brace (and possible following whitespace) optional?
> Considering that the language demands that the brace be on the same
> line, I'd think the brace should be mandatory.
>
I did this to support non-standard formatting. It's a niche case though,
maybe we could only support the standard formatting and modify the doc
to reflect this change.
> I suppose you made whitespace after 'func' optional to be able to
> recognize a method (which hasn't been gofmt'd):
>
> func(x *X) name {
>
> rather than the more typical:
>
> func (x *X) name {
>
> I wonder if it would make sense to tighten the expression to recognize
> functions and methods as distinct cases:
>
> function: mandatory whitespace following 'func'
> method: optional whitespace but mandatory '(' following 'func'
>
> Your current expression could accidentally match:
>
> /*
> Fish like to have
> functors for lunch {
> just like eels}.
> */
>
> but, even the suggested tighter expression could "accidentally" match
> example code in a comment block anyhow, so I guess it probably doesn't
> matter much in practice.
>
Same as before, I did this to support non-standard formatting.
