Haha! Certainly, I wouldn't block a pull because of that. It's just a note that it's not necessary (in case the developer was unclear on when functions must be declared through local variable assignment).

Inside a block of variables, the assignment approach imposes another level of indentation, which I hear some developers in the Open Source community always moaning about. In terms of extra characters, there's gonna be an "=" and a comma (assuming the assignment is inside a var block containing other variable assignments). If many nested functions are declared in this way, there'll be a small bit of weight added. I'm not sure if there are any performance implications of a named function versus an anonymous function assigned to a local variable - I will read up.