SquizMultiLineFunctionDeclaration error when param has trailing comment

[gsouf]

Hi,

I'm encountering issue with Squiz.Functions.MultiLineFunctionDeclaration.OneParamPerLine snif.

The following works perfectly:

    public function __construct(
        $foo,
        $bar
    ) {

while the following wont:

    public function __construct(
        $foo,   // This is foo
        $bar
    ) {

and triggers error - Multi-line function declarations must define one parameter per line (Squiz.Functions.MultiLineFunctionDeclaration.OneParamPerLine)

Please note that the following is working:

    public function __construct(
        $foo /* this is foo */,
        $bar // this is bar
    ) {

[gsherwood]

I think it wants the comma to be the last thing on the line, but comments should be ignored by this check. No idea why the last code block works though. Will investigate when fixing.

[jrfnl]

@gsouf @gsherwood Had a quick look and turned out to be a very simple fix ;-) See PR #1965