Bug 42471 – ES5 allows use of reserved words as IdentifierName

• Component: New Bugs
• Status: RESOLVED FIXED

[Oliver Hunt]

ES5 allows use of reserved words as IdentifierName

[Zoltan Herczeg]

Interesting. Example?

[Oliver Hunt]

Created attachment 61829 [details]
Patch

[Oliver Hunt]

(In reply to comment #1)
> Interesting. Example?

a.var = 5;
a={var:5}

[Geoffrey Garen]

Comment on attachment 61829 [details]
Patch

r=me

[Zoltan Herczeg]

*** Bug 32722 has been marked as a duplicate of this bug. ***

[Joseph Pecoraro]

Thanks for duping Zoltan, and great to see this was fixed Oliver!

I notice Oliver's changelog says "These tests are wrong, unsure how
to get sputnik tests corrected." Were they wrong for ES3, or just
wrong for ES5? It would be nice to have a list of the tests in sputnik
that are expected to fail going from ES3 to ES5.

[Oliver Hunt]

(In reply to comment #6)
> Thanks for duping Zoltan, and great to see this was fixed Oliver!
> 
> I notice Oliver's changelog says "These tests are wrong, unsure how
> to get sputnik tests corrected." Were they wrong for ES3, or just
> wrong for ES5? It would be nice to have a list of the tests in sputnik
> that are expected to fail going from ES3 to ES5.

Sputnik is (afaict) intended to be an ES5 test suite based on other tests.

[Adam Barth]

Comment on attachment 61829 [details]
Patch

Is this patch still current?

[Zoltan Herczeg]

Maybe http://trac.webkit.org/changeset/63566 ?

I close this bug.

[Jeff Walden (remove +bwo to email)]

*** Bug 32721 has been marked as a duplicate of this bug. ***