Changeset 3306 in webkit for trunk/JavaScriptCore/kjs/function.h

Timestamp:

Jan 12, 2003, 10:43:08 AM (22 years ago)

Author:

darin

Message:

Reviewed by John.

• changes inspired by things I noticed reviewing diffs vs. KDE when preparing the tarball

• kjs/function.cpp: (GlobalFuncImp::call): Use strtol when strtoll is not available. Do #ifndef NDEBUG, not #if !NDEBUG.
• kjs/function.h: Do #ifndef NDEBUG, not #if !NDEBUG.
• kjs/internal.cpp: (InterpreterImp::initGlobalObject): Do #ifndef NDEBUG, not #if !NDEBUG. (KJS::printInfo): Remove case for ListType and remove default case that just ends up suppressing the "missing case" warning and does no good.
• kjs/interpreter.cpp: (Interpreter::evaluate): Do #ifndef NDEBUG, not #if !NDEBUG.
• kjs/nodes.cpp: (Node::finalCheck): Fix accidentally-deleted code in an ifdef we never compile. (FunctionCallNode::evaluate): Remove bogus XXX comment. Maciej put this comment in, and together we determined it's not needed. (TypeOfNode::evaluate): Ditto.
• kjs/object.cpp: Remove assert that refers to ListType.
• kjs/value.h: Remove ListType.

File:

• trunk/JavaScriptCore/kjs/function.h (modified) (1 diff)

trunk/JavaScriptCore/kjs/function.h

@@ -129 +129 @@
     virtual CodeType codeType() const;
     enum { Eval, ParseInt, ParseFloat, IsNaN, IsFinite, Escape, UnEscape 
-#if !NDEBUG
+#ifndef NDEBUG
            , KJSPrint
 #endif