In general, it's convenient to put all the available params in the formal parameter list of a function so that when you're writing/modifying it, it's obvious what data is available to work with.   However, this means that function parameters sometimes go unused, which caused old versions of jshint to choke.  However, new versions of jshint allow that behavior to be tuned.

To me, this seems like an obvious win.  However, if you feel that it's non-obvious enough that you'd rather pull it out into a separate PR / card, I guess we can do that.