Why is this change necessary? I know it makes the internal code a little less confusing (having the function and parameter name not being named the same), but if your ddoc pull warns or errors on this style of code then I think the pull should be fixed up. People might want to write code like this without seeing any warnings.