Changeset 28855 in webkit for trunk/JavaScriptCore/kjs/nodes.cpp

Timestamp:

Dec 18, 2007, 11:42:49 PM (17 years ago)

Author:

[email protected]

Message:

Remove dead code due to removal of post-parse declaration discovery.

RS=Geoff.

Due to the removal of the declaration discovery pass after parsing we
no longer need any of the logic used for that discovery.

File:

• trunk/JavaScriptCore/kjs/nodes.cpp (modified) (24 diffs)

trunk/JavaScriptCore/kjs/nodes.cpp

@@ -210 +210 @@
 
 Node::Node()
-    : m_mayHaveDeclarations(false)
-    , m_expectedReturnType(ObjectType)
+    : m_expectedReturnType(ObjectType)
 {
   m_line = lexer().lineNo();
@@ -217 +216 @@
 
 Node::Node(JSType expectedReturn)
-    : m_mayHaveDeclarations(false)
-    , m_expectedReturnType(expectedReturn)
+    : m_expectedReturnType(expectedReturn)
 {
     m_line = lexer().lineNo();
@@ -3451 +3449 @@
     , init(in)
 {
-    m_mayHaveDeclarations = true; 
 }
 
@@ -3460 +3457 @@
     if (init)
         nodeStack.append(init.get());
-}
-
-void VarDeclNode::getDeclarations(DeclarationStacks& stacks)
-{
-    if (next) {
-        ASSERT(next->mayHaveDeclarations());
-        stacks.nodeStack.append(next.get()); 
-    }
-
-    // The normal check to avoid overwriting pre-existing values with variable
-    // declarations doesn't work for the "arguments" property because we 
-    // instantiate it lazily. So we need to check here instead.
-    if (ident == stacks.exec->propertyNames().arguments)
-        return;
-
-    stacks.varStack.append(this); 
 }
 
@@ -3576 +3557 @@
 }
 
-void VarStatementNode::getDeclarations(DeclarationStacks& stacks)
-{
-    ASSERT(next->mayHaveDeclarations());
-    stacks.nodeStack.append(next.get());
-}
-
 // ------------------------------ Helper functions for handling Vectors of StatementNode -------------------------------
 
@@ -3592 +3567 @@
         stack.append((*it).get());
     }
-}
-
-static inline void statementListGetDeclarations(SourceElements& statements, DeclarationStacks& stacks)
-{
-    SourceElements::iterator it = statements.end();
-    SourceElements::iterator begin = statements.begin();
-    while (it != begin) {
-        --it;
-        if ((*it)->mayHaveDeclarations())
-            stacks.nodeStack.append((*it).get());
-    }
-}
-
-static inline Node* statementListInitializeDeclarationStack(SourceElements& statements, DeclarationStacks::NodeStack& stack)
-{
-    ASSERT(!stack.size()); // Otherwise, the removeLast() call might remove someone else's node.
-    
-    SourceElements::iterator it = statements.end();
-    SourceElements::iterator begin = statements.begin();
-    
-    while (it != begin) {
-        --it;
-        if ((*it)->mayHaveDeclarations())
-             stack.append((*it).get());
-    }
-
-    if (!stack.size())
-        return 0;
-
-    Node* n = stack.last();
-    stack.removeLast();
-    return n;
 }
 
@@ -3667 +3610 @@
 {
     ASSERT(m_children);
-    m_mayHaveDeclarations = true; 
 }
 
@@ -3673 +3615 @@
 {
     statementListPushFIFO(*m_children, nodeStack);
-}
-
-void BlockNode::getDeclarations(DeclarationStacks& stacks)
-{ 
-    statementListGetDeclarations(*m_children, stacks);
 }
 
@@ -3745 +3682 @@
 }
 
-void IfNode::getDeclarations(DeclarationStacks& stacks)
-{ 
-    if (statement2 && statement2->mayHaveDeclarations()) 
-        stacks.nodeStack.append(statement2.get()); 
-    if (statement1->mayHaveDeclarations()) 
-        stacks.nodeStack.append(statement1.get()); 
-}
-
 // ------------------------------ DoWhileNode ----------------------------------
 
@@ -3793 +3722 @@
 
     return Completion(); // work around gcc 4.0 bug
-}
-
-void DoWhileNode::getDeclarations(DeclarationStacks& stacks)
-{ 
-    if (statement->mayHaveDeclarations()) 
-        stacks.nodeStack.append(statement.get()); 
 }
 
@@ -3841 +3764 @@
 
     return Completion(); // work around gcc 4.0 bug
-}
-
-void WhileNode::getDeclarations(DeclarationStacks& stacks)
-{ 
-    if (statement->mayHaveDeclarations()) 
-        stacks.nodeStack.append(statement.get()); 
 }
 
@@ -3904 +3821 @@
 }
 
-void ForNode::getDeclarations(DeclarationStacks& stacks)
-{ 
-    if (statement->mayHaveDeclarations()) 
-        stacks.nodeStack.append(statement.get()); 
-    if (expr1 && expr1->mayHaveDeclarations()) 
-        stacks.nodeStack.append(expr1.get()); 
-}
-
 // ------------------------------ ForInNode ------------------------------------
 
@@ -3917 +3826 @@
   : init(0L), lexpr(l), expr(e), varDecl(0L), statement(s)
 {
-    m_mayHaveDeclarations = true; 
 }
 
@@ -3923 +3831 @@
   : ident(i), init(in), expr(e), statement(s)
 {
-  m_mayHaveDeclarations = true; 
-
   // for( var foo = bar in baz )
   varDecl = new VarDeclNode(ident, init.get(), VarDeclNode::Variable);
@@ -3937 +3843 @@
     if (varDecl)
         nodeStack.append(varDecl.get());
-}
-
-void ForInNode::getDeclarations(DeclarationStacks& stacks)
-{ 
-    if (statement->mayHaveDeclarations()) 
-        stacks.nodeStack.append(statement.get()); 
-    if (varDecl && varDecl->mayHaveDeclarations()) 
-        stacks.nodeStack.append(varDecl.get()); 
 }
 
@@ -4103 +4001 @@
 // ------------------------------ WithNode -------------------------------------
 
-void WithNode::getDeclarations(DeclarationStacks& stacks)
-{ 
-    if (statement->mayHaveDeclarations()) 
-        stacks.nodeStack.append(statement.get()); 
-}
-
 void WithNode::optimizeVariableAccess(SymbolTable&, DeclarationStacks::NodeStack& nodeStack)
 {
@@ -4141 +4033 @@
 }
 
-void CaseClauseNode::getDeclarations(DeclarationStacks& stacks)
-{ 
-    if (m_children) 
-        statementListGetDeclarations(*m_children, stacks);
-}
-
 // ECMA 12.11
 JSValue *CaseClauseNode::evaluate(ExecState *exec)
@@ -4174 +4060 @@
 }
 
-void ClauseListNode::getDeclarations(DeclarationStacks& stacks)
-{ 
-    if (next && next->mayHaveDeclarations()) 
-        stacks.nodeStack.append(next.get()); 
-    if (clause->mayHaveDeclarations()) 
-        stacks.nodeStack.append(clause.get()); 
-}
-
 // ------------------------------ CaseBlockNode --------------------------------
 
@@ -4189 +4067 @@
     , list2(l2)
 {
-    m_mayHaveDeclarations = true; 
 }
  
@@ -4200 +4077 @@
     if (list1)
         nodeStack.append(list1.get());
-}
-
-void CaseBlockNode::getDeclarations(DeclarationStacks& stacks) 
-{ 
-    if (list2 && list2->mayHaveDeclarations()) 
-        stacks.nodeStack.append(list2.get());
-    if (def && def->mayHaveDeclarations()) 
-        stacks.nodeStack.append(def.get()); 
-    if (list1 && list1->mayHaveDeclarations()) 
-        stacks.nodeStack.append(list1.get()); 
 }
 
@@ -4283 +4150 @@
 }
 
-void SwitchNode::getDeclarations(DeclarationStacks& stacks) 
-{ 
-    if (block->mayHaveDeclarations()) 
-        stacks.nodeStack.append(block.get()); 
-}
-
 // ECMA 12.11
 Completion SwitchNode::execute(ExecState *exec)
@@ -4311 +4172 @@
 {
     nodeStack.append(statement.get());
-}
-
-void LabelNode::getDeclarations(DeclarationStacks& stacks) 
-{ 
-    if (statement->mayHaveDeclarations()) 
-        stacks.nodeStack.append(statement.get()); 
 }
 
@@ -4359 +4214 @@
         nodeStack.append(finallyBlock.get());
     nodeStack.append(tryBlock.get());
-}
-
-void TryNode::getDeclarations(DeclarationStacks& stacks) 
-{ 
-    if (finallyBlock && finallyBlock->mayHaveDeclarations()) 
-        stacks.nodeStack.append(finallyBlock.get()); 
-    if (catchBlock && catchBlock->mayHaveDeclarations()) 
-        stacks.nodeStack.append(catchBlock.get()); 
-    if (tryBlock->mayHaveDeclarations()) 
-        stacks.nodeStack.append(tryBlock.get()); 
 }
 
@@ -4612 +4457 @@
 }
 
-void FuncDeclNode::getDeclarations(DeclarationStacks& stacks) 
-{
-    stacks.functionStack.append(this);
-}
-
 FunctionImp* FuncDeclNode::makeFunction(ExecState* exec)
 {