Changeset 187760 in webkit for trunk/Source/JavaScriptCore/parser/Parser.cpp

Timestamp:

Aug 3, 2015, 1:47:54 PM (10 years ago)

Author:

[email protected]

Message:

Clean up the naming for AST expression generation.
​https://bugs.webkit.org/show_bug.cgi?id=147581

Patch by Keith Miller <​[email protected]> on 2015-08-03
Reviewed by Yusuke Suzuki.

• parser/ASTBuilder.h:

(JSC::ASTBuilder::createThisExpr):
(JSC::ASTBuilder::createSuperExpr):
(JSC::ASTBuilder::createNewTargetExpr):
(JSC::ASTBuilder::thisExpr): Deleted.
(JSC::ASTBuilder::superExpr): Deleted.
(JSC::ASTBuilder::newTargetExpr): Deleted.

• parser/Parser.cpp:

(JSC::Parser<LexerType>::parsePrimaryExpression):
(JSC::Parser<LexerType>::parseMemberExpression):

• parser/SyntaxChecker.h:

(JSC::SyntaxChecker::createThisExpr):
(JSC::SyntaxChecker::createSuperExpr):
(JSC::SyntaxChecker::createNewTargetExpr):
(JSC::SyntaxChecker::thisExpr): Deleted.
(JSC::SyntaxChecker::superExpr): Deleted.
(JSC::SyntaxChecker::newTargetExpr): Deleted.

File:

• trunk/Source/JavaScriptCore/parser/Parser.cpp (modified) (3 diffs)

trunk/Source/JavaScriptCore/parser/Parser.cpp

@@ -2767 +2767 @@
         JSTokenLocation location(tokenLocation());
         next();
-        return context.thisExpr(location, m_thisTDZMode);
+        return context.createThisExpr(location, m_thisTDZMode);
     }
     case IDENT: {
@@ -2917 +2917 @@
                 semanticFailIfFalse(currentScope()->isFunction(), "new.target is only valid inside functions");
                 baseIsNewTarget = true;
-                base = context.newTargetExpr(location);
+                base = context.createNewTargetExpr(location);
                 newCount--;
                 next();
@@ -2927 +2927 @@
 
     if (baseIsSuper) {
-        base = context.superExpr(location);
+        base = context.createSuperExpr(location);
         next();
         currentScope()->setNeedsSuperBinding();