708892 - Clean up destructuring var initializers

Status: RESOLVED INVALID

(Core :: JavaScript Engine, defect)

Description

[David Mandelin [:dmandelin]]

Clean up the code for destructuring var initializers, taking care to make all the front-end attributes propagate correctly.

Comment 1

[Jeff Walden [:Waldo]]

The patch I had written, that was down to half a dozen failures, conflicts horribly with bug 692274's patch, which landed a week or so ago.  It's going to take some time to sort that out.  I knew I shouldn't have believed Luke when he said the two would be orthogonal.  :-P

Comment 2

[Luke Wagner [:luke]]

Well, logically orthogonal ;)

Comment 3

[Al Billings [:abillings - ex-MoCo]]

What's the current status on this work?

Comment 4

[Jeff Walden [:Waldo]]

I haven't touched it since about comment 1.  I won't have time to resurrect it until I finish bug 586842, which has been in-progress for way longer than I ever hoped or wanted, and which is a must-fix so that it can be integrated in Ionmonkey, which is progressing along quite well at this point and which would be better off doing not too much work that will only be invalidated when that bug's fixed.

Comment 5

[André Bargull [:anba]]

Unlikely to be still valid, given that destructuring is now part of ECMAScript proper and the destructuring parser part has been rewritten (at least once).