Changeset 26690 in webkit for trunk/JavaScriptCore/kjs/function.cpp

Timestamp:

Oct 16, 2007, 4:35:01 PM (18 years ago)

Author:

ggaren

Message:

Reviewed by Darin Adler.

Removed KJS_VERBOSE because it was getting in the way of readability,
and the messages didn't seem very helpful.

• kjs/function.cpp: (KJS::FunctionImp::callAsFunction): (KJS::FunctionImp::passInParameters):
• kjs/lookup.h: (KJS::lookupPut):
• kjs/object.cpp: (KJS::JSObject::put):
• kjs/value.h:

File:

• trunk/JavaScriptCore/kjs/function.cpp (modified) (2 diffs)

trunk/JavaScriptCore/kjs/function.cpp

@@ -107 +107 @@
     comp = Completion(Throw, newExec.exception());
 
-#ifdef KJS_VERBOSE
-  if (comp.complType() == Throw)
-    printInfo(exec,"throwing", comp.value());
-  else if (comp.complType() == ReturnValue)
-    printInfo(exec,"returning", comp.value());
-  else
-    fprintf(stderr, "returning: undefined\n");
-#endif
-
   // The debugger may have been deallocated by now if the WebFrame
   // we were running in has been destroyed, so refetch it.
@@ -152 +143 @@
     JSObject* variable = exec->context()->variableObject();
 
-#ifdef KJS_VERBOSE
-    fprintf(stderr, "---------------------------------------------------\n"
-          "processing parameters for %s call\n",
-          functionName().isEmpty() ? "(internal)" : functionName().ascii());
-#endif
-
     size_t size = parameters.size();
     for (size_t i = 0; i < size; ++i) {
-#ifdef KJS_VERBOSE
-      fprintf(stderr, "setting parameter %s ", parameters.at(i).name.ascii());
-      printInfo(exec, "to", args[i]);
-#endif
       variable->put(exec, parameters[i], args[i], DontDelete);
     }