node-xml icon indicating copy to clipboard operation
node-xml copied to clipboard

Removed confusing use of new Array()

Open coolaj86 opened this issue 12 years ago • 0 comments

I argue that [] should be used instead of new Array(), and especially instead of new Array(arg0, arg1, ...) to avoid confusion (i.e. I was confused when I saw it).

I wasn't sure if there would be a bug or not in the case that name was a number and value was undefined. According to Firefox, it should never result in a bug. I assume Chrome behaves the same way.

https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Global_Objects/Array

coolaj86 avatar May 27 '13 16:05 coolaj86