max_function_length counts multiple function clauses as a single function

[kivra-pauoli]

Bug Description

Note: might not be a bug.

max_function_length complains if a function definition spans over a given number of lines (by default: 30)...

... but it's also taking into account the multiple function clauses. So 15 clauses with 1 line each, in the body, would count for 30 lines. Is this expected?

To Reproduce

16 function clauses with 2 line (definition) each, 1 line for head, 1 line for body.

Expected Behavior

? no warning, maybe?

rebar3 Logs

Not applicable.

Additional Context

Not relevant.

[elbrujohalcon]

Yeah, this rule was always a bit… blunt.
If we're going to adjust it, then it will need a rename as max_function_clause_length, I guess.

[kivra-pauoli]

Or similar, yes. For the time being I'm Ok with just adding an exception in my rules, ofc. This was mostly just me wondering if we got (from the parsing) the size of each clause or not...