Discuss
  • Could this be removed from Array#concat.

Could this be removed from Array#concat.

# John-David Dalton (16 years ago)

I think that under 15.4.4.4 Array.prototype.concat 5 -> b. -> iii -> 3 - > "a. Call ToString(n)." could be removed as it

is performed in c.

  • John-David Dalton
# Thomas L. Shinnick (16 years ago)

At 10:29 PM 6/14/2009, John-David Dalton wrote:

I think that under 15.4.4.4 Array.prototype.concat 5 -> b. -> iii -> 3 - > "a. Call ToString(n)." could be removed as it is performed in c.

  • John-David Dalton

I had also thought this in my comments to the editors (apparently missed) prior to publishing, saying Also, step 5.b.iii.4 could probably be removed, as the call to [[DefineOwnProperty]] does its own ToString(n)

( before some late reorganization "5.b.iii.4" was the current 5.b.iii.3.a )

So... seconded. (Or firsted?... anyway agreed :-) )

# Allen Wirfs-Brock (16 years ago)

Also agree and will fix it. I haven't worked though all these sorts of editorial comments yet...

From: es-discuss-bounces at mozilla.org [mailto:es-discuss-bounces at mozilla.org] On Behalf Of Thomas L. Shinnick Sent: Sunday, June 14, 2009 8:46 PM To: es-discuss at mozilla.org Subject: Re: Could this be removed from Array#concat.

At 10:29 PM 6/14/2009, John-David Dalton wrote:

I think that under 15.4.4.4 Array.prototype.concat 5 -> b. -> iii -> 3 - > "a. Call ToString(n)." could be removed as it

is performed in c.

  • John-David Dalton

I had also thought this in my comments to the editors (apparently missed) prior to publishing, saying Also, step 5.b.iii.4 could probably be removed, as the call to [[DefineOwnProperty]] does its own ToString(n)

( before some late reorganization "5.b.iii.4" was the current 5.b.iii.3.a )

So... seconded. (Or firsted?... anyway agreed :-) )