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

Timestamp:

Sep 8, 2005, 6:19:19 PM (20 years ago)

Author:

darin

Message:

• fixed overloaded versions of throwError so that they substitute *all* expected parameters into the message string -- some versions used to skip parameters, resulting in "%s" being printed in the error message.

Reviewed by Geoff.

• kjs/nodes.h: Updated declarations to use "const &" and not to name parameters
• kjs/nodes.cpp: (Node::throwError): Updated to match above and add one missing call to substitute.

File:

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

trunk/JavaScriptCore/kjs/nodes.cpp

@@ -147 +147 @@
 
 
-ValueImp *Node::throwError(ExecState *exec, ErrorType e, const char *msg, Identifier label)
+ValueImp *Node::throwError(ExecState *exec, ErrorType e, const char *msg, const Identifier &label)
 {
     UString message = msg;
@@ -163 +163 @@
 }
 
-ValueImp *Node::throwError(ExecState *exec, ErrorType e, const char *msg, ValueImp *v, Node *expr, Identifier label)
+ValueImp *Node::throwError(ExecState *exec, ErrorType e, const char *msg, ValueImp *v, Node *expr, const Identifier &label)
 {
     UString message = msg;
     substitute(message, v->toString(exec));
     substitute(message, expr->toString());
+    substitute(message, label.ustring());
     return KJS::throwError(exec, e, message, lineNo(), sourceId(), &sourceURL);
 }
 
-ValueImp *Node::throwError(ExecState *exec, ErrorType e, const char *msg, ValueImp *v, Identifier label)
+ValueImp *Node::throwError(ExecState *exec, ErrorType e, const char *msg, ValueImp *v, const Identifier &label)
 {
     UString message = msg;
@@ -178 +179 @@
     return KJS::throwError(exec, e, message, lineNo(), sourceId(), &sourceURL);
 }
-
 
 void Node::setExceptionDetailsIfNeeded(ExecState *exec)