Array concat has no error in < IE8 specific for sparse array

andrewplummer / Sugar

A Javascript library for working with native objects.

https://sugarjs.com/

MIT License

4.53k stars 305 forks source link

Array concat has no error in < IE8 specific for sparse array #641

Closed lihroff closed 5 years ago

lihroff commented 5 years ago

Below is some test showed both in IE7 & 5:

and the HAS_CONCAT_BUG method showed in IE7 & 5:

I found that this method can't prove anything. Although IE7&5 will print out the wrong result but the results obtained by index access and length attribute are correct.

andrewplummer commented 5 years ago

Sorry for the delay here. Array.concat is actually native JS and has nothing to do with Sugar, in fact it's available from ES3 so Sugar isn't even providing a polyfill here.

Please feel free to reopen this if you think there's another issue here.