Changeset 18837 in webkit for trunk/JavaScriptCore/kjs/Parser.cpp

Timestamp:

Jan 13, 2007, 11:53:38 PM (18 years ago)

Author:

bdash

Message:

2007-01-14 Mark Rowe <​[email protected]>

Reviewed by Mitz.

Minor fixes to JavaScript pretty-printing.

• JavaScriptCore.exp:
• kjs/Parser.cpp: (KJS::Parser::prettyPrint): Return line number and error message if parsing fails.
• kjs/Parser.h:
• kjs/nodes2string.cpp: (ElementNode::streamTo): Include comma delimiters in array literals. (PropertyNameNode::streamTo): Quote property names in object literals to handle the case when the property name is not a valid identifier.
• kjs/testkjs.cpp: (doIt): Print any errors encountered while pretty-printing.

2007-01-14 Mark Rowe <​[email protected]>

Reviewed by Mitz.

Layout tests for fixes to JavaScript pretty-printing.

• fast/js/pretty-print-expected.txt:
• fast/js/resources/pretty-print.js: Test handling of object literal with non-identifier property name, and of array literals.

File:

• trunk/JavaScriptCore/kjs/Parser.cpp (modified) (1 diff)

trunk/JavaScriptCore/kjs/Parser.cpp

@@ -110 +110 @@
 }
 
-UString Parser::prettyPrint(const UString& code)
+UString Parser::prettyPrint(const UString& code, int* errLine, UString* errMsg)
 {
-    RefPtr<ProgramNode> progNode = parse(UString(), 0, code.data(), code.size());
+    RefPtr<ProgramNode> progNode = parse(UString(), 0, code.data(), code.size(), 0, errLine, errMsg);
     if (!progNode)
         return 0;