Style/SymbolProc auto-correct breaking syntax

[NobodysNightmare]

Reproduction

$ ruby -v
ruby 2.2.3p173 (2015-08-18 revision 51636) [x86_64-linux]
$ rubocop -v
0.35.1

Using the following test file (dubbed rubocop_test.rb):

def mail(to:, subject:)
  yield to
end

mail(
      to: 'foo',
      subject: 'bar',
    ) { |format| format.text }

Run rubocop -a rubocop_test.rb

Expected

A valid ruby file, with fewer offenes.

Actual

The syntax gets messed up by the Style/SymbolProc rule:

def mail(to:, subject:)
  yield to
end

mail(
  to: 'foo',
  subject: 'bar', &:text,
)

The extra comma after the &:text parameter is breaking the syntax.

[alexdowad]

The extra comma after the &:text parameter is breaking the syntax.

The extra comma is actually the one which was already there, after subject: 'bar'.

Thinking about how to resolve this...

[alexdowad]

I have an idea.

[NobodysNightmare]

Wow...

I totally copy and pasted that example out of production code and didn't notice the comma after subject: 'bar'...
Now I am wondering why it was valid before, but I guess ruby is forgiving at times...

[alexdowad]

Just pushing a fix to my open PR.

[NobodysNightmare]

@alexdowad Wow, that is what I call a quick response!

Thanks for your efforts 👍 (and bonus points for the title of your PR ^^)