Changeset 31072 in webkit for trunk/JavaScriptCore/kjs/Parser.h

Timestamp:

Mar 14, 2008, 6:05:55 PM (17 years ago)

Author:

[email protected]

Message:

Add logic to track whether a function uses a locally scoped eval or requires a closure

Reviewed by Maciej

Now that we limit eval we can track those uses of eval that operate
in the local scope and functions that require a closure. We track
this information during initial parsing to avoid yet another tree
walk.

File:

• trunk/JavaScriptCore/kjs/Parser.h (modified) (3 diffs)

trunk/JavaScriptCore/kjs/Parser.h

@@ -53 +53 @@
 
         void didFinishParsing(SourceElements*, ParserRefCountedData<DeclarationStacks::VarStack>*, 
-                              ParserRefCountedData<DeclarationStacks::FunctionStack>*, int lastLine);
+                              ParserRefCountedData<DeclarationStacks::FunctionStack>*, bool usesEval, bool needsClosure, int lastLine);
 
     private:
@@ -67 +67 @@
         RefPtr<ParserRefCountedData<DeclarationStacks::VarStack> > m_varDeclarations;
         RefPtr<ParserRefCountedData<DeclarationStacks::FunctionStack> > m_funcDeclarations;
+        bool m_usesEval;
+        bool m_needsClosure;
         int m_lastLine;
     };
@@ -85 +87 @@
         RefPtr<ParsedNode> node = ParsedNode::create(m_sourceElements.release().get(),
                                                      m_varDeclarations ? &m_varDeclarations->data : 0, 
-                                                     m_funcDeclarations ? &m_funcDeclarations->data : 0);
+                                                     m_funcDeclarations ? &m_funcDeclarations->data : 0,
+                                                     m_usesEval,
+                                                     m_needsClosure);
         m_varDeclarations = 0;
         m_funcDeclarations = 0;