I consider it bad form to modify arguments in JavaScript, as objects are passed by reference, but primitives are not.  Being consistent about this, makes for less confusing code.