Changeset 192768 in webkit for trunk/Source/JavaScriptCore/parser/SyntaxChecker.h

Timestamp:

Nov 24, 2015, 5:43:14 PM (9 years ago)

Author:

[email protected]

Message:

[JSC] support Computed Property Names in destructuring Patterns
​https://bugs.webkit.org/show_bug.cgi?id=151494

Patch by Caitlin Potter <​[email protected]> on 2015-11-24
Reviewed by Saam Barati.

Add support for computed property names in destructuring BindingPatterns
and AssignmentPatterns.

Productions BindingProperty(1) and AssignmentProperty(2) allow for any valid
PropertName(3), including ComputedPropertyName(4)

1: ​http://tc39.github.io/ecma262/#prod-BindingProperty
2: ​http://tc39.github.io/ecma262/#prod-AssignmentProperty
3: ​http://tc39.github.io/ecma262/#prod-PropertyName
4: ​http://tc39.github.io/ecma262/#prod-ComputedPropertyName

• bytecompiler/NodesCodegen.cpp:

(JSC::ObjectPatternNode::bindValue):

• parser/ASTBuilder.h:

(JSC::ASTBuilder::appendObjectPatternEntry):

• parser/Nodes.h:

(JSC::ObjectPatternNode::appendEntry):

• parser/Parser.cpp:

(JSC::Parser<LexerType>::parseDestructuringPattern):

• parser/SyntaxChecker.h:

(JSC::SyntaxChecker::operatorStackPop):

• tests/es6.yaml:
• tests/es6/destructuring_assignment_computed_properties.js: Added.

(test):
(test.computeName):
(test.loadValue):
(test.out.get a):
(test.out.set a):
(test.out.get b):
(test.out.set b):
(test.out.get c):
(test.out.set c):
(test.get var):

File:

• trunk/Source/JavaScriptCore/parser/SyntaxChecker.h (modified) (1 diff)

trunk/Source/JavaScriptCore/parser/SyntaxChecker.h

@@ -344 +344 @@
     {
     }
+    void appendObjectPatternEntry(ArrayPattern, const JSTokenLocation&, Expression, DestructuringPattern, Expression)
+    {
+    }
+
     DestructuringPattern createBindingLocation(const JSTokenLocation&, const Identifier&, const JSTextPosition&, const JSTextPosition&, AssignmentContext)
     {