Reviewed by Beth Dakin.
authorweinig@apple.com <weinig@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 2 Jan 2008 21:32:39 +0000 (21:32 +0000)
committerweinig@apple.com <weinig@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 2 Jan 2008 21:32:39 +0000 (21:32 +0000)
        Cleanup error_object.h/cpp.

        * kjs/JSGlobalObject.cpp:
        (KJS::JSGlobalObject::reset):
        * kjs/error_object.cpp:
        (KJS::):
        (KJS::ErrorInstance::ErrorInstance):
        (KJS::ErrorPrototype::ErrorPrototype):
        (KJS::ErrorProtoFuncToString::ErrorProtoFuncToString):
        (KJS::ErrorProtoFuncToString::callAsFunction):
        (KJS::ErrorObjectImp::ErrorObjectImp):
        (KJS::ErrorObjectImp::implementsConstruct):
        (KJS::ErrorObjectImp::construct):
        (KJS::ErrorObjectImp::callAsFunction):
        (KJS::NativeErrorPrototype::NativeErrorPrototype):
        (KJS::NativeErrorImp::NativeErrorImp):
        (KJS::NativeErrorImp::implementsConstruct):
        (KJS::NativeErrorImp::construct):
        (KJS::NativeErrorImp::callAsFunction):
        (KJS::NativeErrorImp::mark):
        * kjs/error_object.h:
        (KJS::ErrorInstance::classInfo):
        (KJS::NativeErrorImp::classInfo):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29091 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JavaScriptCore/ChangeLog
JavaScriptCore/kjs/JSGlobalObject.cpp
JavaScriptCore/kjs/error_object.cpp
JavaScriptCore/kjs/error_object.h

index fab4981341d31f34882e0c28574a6cf95d0b411f..cecea19963d60624b78bdfd5fffd7eb1b03d7b02 100644 (file)
@@ -1,3 +1,31 @@
+2008-01-02  Sam Weinig  <sam@webkit.org>
+
+        Reviewed by Beth Dakin.
+
+        Cleanup error_object.h/cpp.
+
+        * kjs/JSGlobalObject.cpp:
+        (KJS::JSGlobalObject::reset):
+        * kjs/error_object.cpp:
+        (KJS::):
+        (KJS::ErrorInstance::ErrorInstance):
+        (KJS::ErrorPrototype::ErrorPrototype):
+        (KJS::ErrorProtoFuncToString::ErrorProtoFuncToString):
+        (KJS::ErrorProtoFuncToString::callAsFunction):
+        (KJS::ErrorObjectImp::ErrorObjectImp):
+        (KJS::ErrorObjectImp::implementsConstruct):
+        (KJS::ErrorObjectImp::construct):
+        (KJS::ErrorObjectImp::callAsFunction):
+        (KJS::NativeErrorPrototype::NativeErrorPrototype):
+        (KJS::NativeErrorImp::NativeErrorImp):
+        (KJS::NativeErrorImp::implementsConstruct):
+        (KJS::NativeErrorImp::construct):
+        (KJS::NativeErrorImp::callAsFunction):
+        (KJS::NativeErrorImp::mark):
+        * kjs/error_object.h:
+        (KJS::ErrorInstance::classInfo):
+        (KJS::NativeErrorImp::classInfo):
+
 2008-01-02  Mark Rowe  <mrowe@apple.com>
 
         Rubber-stamped by Alp Toker.
index 7a188967049b3badcc77f5501139976a4d99fe75..9679c147f6f0a624cea84b01810a68bb6569cb92 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2007 Apple Inc.  All rights reserved.
+ * Copyright (C) 2007, 2008 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -213,12 +213,12 @@ void JSGlobalObject::reset(JSValue* prototype)
     d()->regExpPrototype = new RegExpPrototype(exec, d()->objectPrototype, d()->functionPrototype);;
     d()->errorPrototype = new ErrorPrototype(exec, d()->objectPrototype, d()->functionPrototype);
     
-    d()->evalErrorPrototype = new NativeErrorPrototype(exec, d()->errorPrototype, EvalError, "EvalError", "EvalError");
-    d()->rangeErrorPrototype = new NativeErrorPrototype(exec, d()->errorPrototype, RangeError, "RangeError", "RangeError");
-    d()->referenceErrorPrototype = new NativeErrorPrototype(exec, d()->errorPrototype, ReferenceError, "ReferenceError", "ReferenceError");
-    d()->syntaxErrorPrototype = new NativeErrorPrototype(exec, d()->errorPrototype, SyntaxError, "SyntaxError", "SyntaxError");
-    d()->typeErrorPrototype = new NativeErrorPrototype(exec, d()->errorPrototype, TypeError, "TypeError", "TypeError");
-    d()->URIErrorPrototype = new NativeErrorPrototype(exec, d()->errorPrototype, URIError, "URIError", "URIError");
+    d()->evalErrorPrototype = new NativeErrorPrototype(exec, d()->errorPrototype, "EvalError", "EvalError");
+    d()->rangeErrorPrototype = new NativeErrorPrototype(exec, d()->errorPrototype, "RangeError", "RangeError");
+    d()->referenceErrorPrototype = new NativeErrorPrototype(exec, d()->errorPrototype, "ReferenceError", "ReferenceError");
+    d()->syntaxErrorPrototype = new NativeErrorPrototype(exec, d()->errorPrototype, "SyntaxError", "SyntaxError");
+    d()->typeErrorPrototype = new NativeErrorPrototype(exec, d()->errorPrototype, "TypeError", "TypeError");
+    d()->URIErrorPrototype = new NativeErrorPrototype(exec, d()->errorPrototype, "URIError", "URIError");
 
     // Constructors
     d()->objectConstructor = new ObjectObjectImp(exec, d()->objectPrototype, d()->functionPrototype);
index 727a320a75596901fd70b6c497265dfd67fbdb98..ad2a416acb06b00a411aa53fb2c43a99945d5895 100644 (file)
@@ -1,7 +1,6 @@
-// -*- c-basic-offset: 2 -*-
 /*
  *  Copyright (C) 1999-2000 Harri Porten (porten@kde.org)
- *  Copyright (C) 2003, 2007 Apple Inc.
+ *  Copyright (C) 2003, 2008 Apple Inc. All rights reserved.
  *
  *  This library is free software; you can redistribute it and/or
  *  modify it under the terms of the GNU Lesser General Public
 #include "types.h"
 #include "value.h"
 
-using namespace KJS;
+namespace KJS {
 
 // ------------------------------ ErrorInstance ----------------------------
 
-const ClassInfo ErrorInstance::info = {"Error", 0, 0};
+const ClassInfo ErrorInstance::info = { "Error", 0, 0 };
 
-ErrorInstance::ErrorInstance(JSObject *proto)
-: JSObject(proto)
+ErrorInstance::ErrorInstance(JSObject* prototype)
+    : JSObject(prototype)
 {
 }
 
@@ -43,121 +42,120 @@ ErrorInstance::ErrorInstance(JSObject *proto)
 
 // ECMA 15.9.4
 ErrorPrototype::ErrorPrototype(ExecState* exec, ObjectPrototype* objectProto, FunctionPrototype* funcProto)
-  : JSObject(objectProto)
+    : JSObject(objectProto)
 {
-  // The constructor will be added later in ErrorObjectImp's constructor
+    // The constructor will be added later in ErrorObjectImp's constructor
 
-  put(exec, exec->propertyNames().name,     jsString("Error"), DontEnum);
-  put(exec, exec->propertyNames().message,  jsString("Unknown error"), DontEnum);
-  putDirectFunction(new ErrorProtoFunc(exec, funcProto, exec->propertyNames().toString), DontEnum);
+    put(exec, exec->propertyNames().name, jsString("Error"), DontEnum);
+    put(exec, exec->propertyNames().message, jsString("Unknown error"), DontEnum);
+    putDirectFunction(new ErrorProtoFuncToString(exec, funcProto), DontEnum);
 }
 
 // ------------------------------ ErrorProtoFunc ----------------------------
 
-ErrorProtoFunc::ErrorProtoFunc(ExecState* exec, FunctionPrototype* funcProto, const Identifier& name)
-  : InternalFunctionImp(funcProto, name)
+ErrorProtoFuncToString::ErrorProtoFuncToString(ExecState* exec, FunctionPrototype* funcProto)
+    : InternalFunctionImp(funcProto, exec->propertyNames().toString)
 {
-  putDirect(exec->propertyNames().length, jsNumber(0), DontDelete|ReadOnly|DontEnum);
+    putDirect(exec->propertyNames().length, jsNumber(0), DontDelete|ReadOnly|DontEnum);
 }
 
-JSValue* ErrorProtoFunc::callAsFunction(ExecState* exec, JSObject* thisObj, const List &/*args*/)
+JSValue* ErrorProtoFuncToString::callAsFunction(ExecState* exec, JSObject* thisObj, const List&)
 {
-  // toString()
-  UString s = "Error";
+    UString s = "Error";
 
-  JSValue* v = thisObj->get(exec, exec->propertyNames().name);
-  if (!v->isUndefined()) {
-    s = v->toString(exec);
-  }
+    JSValue* v = thisObj->get(exec, exec->propertyNames().name);
+    if (!v->isUndefined())
+        s = v->toString(exec);
 
-  v = thisObj->get(exec, exec->propertyNames().message);
-  if (!v->isUndefined()) {
-    s += ": " + v->toString(exec); // Mozilla compatible format
-  }
+    v = thisObj->get(exec, exec->propertyNames().message);
+    if (!v->isUndefined())
+        // Mozilla compatible format
+        s += ": " + v->toString(exec);
 
-  return jsString(s);
+    return jsString(s);
 }
 
 // ------------------------------ ErrorObjectImp -------------------------------
 
 ErrorObjectImp::ErrorObjectImp(ExecState* exec, FunctionPrototype* funcProto, ErrorPrototype* errorProto)
-  : InternalFunctionImp(funcProto)
+    : InternalFunctionImp(funcProto)
 {
-  // ECMA 15.11.3.1 Error.prototype
-  putDirect(exec->propertyNames().prototype, errorProto, DontEnum|DontDelete|ReadOnly);
-  putDirect(exec->propertyNames().length, jsNumber(1), DontDelete|ReadOnly|DontEnum);
-  //putDirect(namePropertyName, jsString(n));
+    // ECMA 15.11.3.1 Error.prototype
+    putDirect(exec->propertyNames().prototype, errorProto, DontEnum|DontDelete|ReadOnly);
+    putDirect(exec->propertyNames().length, jsNumber(1), DontDelete|ReadOnly|DontEnum);
+    //putDirect(namePropertyName, jsString(n));
 }
 
 bool ErrorObjectImp::implementsConstruct() const
 {
-  return true;
+    return true;
 }
 
 // ECMA 15.9.3
-JSObject* ErrorObjectImp::construct(ExecState* exec, const List &args)
+JSObject* ErrorObjectImp::construct(ExecState* exec, const Listargs)
 {
-  JSObject* proto = static_cast<JSObject*>(exec->lexicalGlobalObject()->errorPrototype());
-  JSObject* imp = new ErrorInstance(proto);
-  JSObject* obj(imp);
+    JSObject* proto = static_cast<JSObject*>(exec->lexicalGlobalObject()->errorPrototype());
+    JSObject* imp = new ErrorInstance(proto);
+    JSObject* obj(imp);
 
-  if (!args[0]->isUndefined())
-    imp->putDirect(exec->propertyNames().message, jsString(args[0]->toString(exec)));
+    if (!args[0]->isUndefined())
+        imp->putDirect(exec->propertyNames().message, jsString(args[0]->toString(exec)));
 
-  return obj;
+    return obj;
 }
 
 // ECMA 15.9.2
-JSValue *ErrorObjectImp::callAsFunction(ExecState *exec, JSObject* /*thisObj*/, const List &args)
+JSValue* ErrorObjectImp::callAsFunction(ExecState* exec, JSObject* /*thisObj*/, const List& args)
 {
-  // "Error()" gives the sames result as "new Error()"
-  return construct(exec,args);
+    // "Error()" gives the sames result as "new Error()"
+    return construct(exec, args);
 }
 
 // ------------------------------ NativeErrorPrototype ----------------------
 
-NativeErrorPrototype::NativeErrorPrototype(ExecState* exec, ErrorPrototype* errorProto, ErrorType et, UString name, UString message)
-  : JSObject(errorProto)
+NativeErrorPrototype::NativeErrorPrototype(ExecState* exec, ErrorPrototype* errorProto, UString name, UString message)
+    : JSObject(errorProto)
 {
-  errType = et;
-  putDirect(exec->propertyNames().name, jsString(name), 0);
-  putDirect(exec->propertyNames().message, jsString(message), 0);
+    putDirect(exec->propertyNames().name, jsString(name), 0);
+    putDirect(exec->propertyNames().message, jsString(message), 0);
 }
 
 // ------------------------------ NativeErrorImp -------------------------------
 
-const ClassInfo NativeErrorImp::info = {"Function", &InternalFunctionImp::info, 0};
+const ClassInfo NativeErrorImp::info = { "Function", &InternalFunctionImp::info, 0 };
 
 NativeErrorImp::NativeErrorImp(ExecState* exec, FunctionPrototype* funcProto, JSObject* prot)
-  : InternalFunctionImp(funcProto)
-  , proto(prot)
+    : InternalFunctionImp(funcProto)
+    , proto(prot)
 {
-  putDirect(exec->propertyNames().length, jsNumber(1), DontDelete|ReadOnly|DontEnum); // ECMA 15.11.7.5
-  putDirect(exec->propertyNames().prototype, proto, DontDelete|ReadOnly|DontEnum);
+    putDirect(exec->propertyNames().length, jsNumber(1), DontDelete|ReadOnly|DontEnum); // ECMA 15.11.7.5
+    putDirect(exec->propertyNames().prototype, proto, DontDelete|ReadOnly|DontEnum);
 }
 
 bool NativeErrorImp::implementsConstruct() const
 {
-  return true;
+    return true;
 }
 
 JSObject* NativeErrorImp::construct(ExecState* exec, const List& args)
 {
-  JSObject* imp = new ErrorInstance(proto);
-  JSObject* obj(imp);
-  if (!args[0]->isUndefined())
-    imp->putDirect(exec->propertyNames().message, jsString(args[0]->toString(exec)));
-  return obj;
+    JSObject* imp = new ErrorInstance(proto);
+    JSObject* obj(imp);
+    if (!args[0]->isUndefined())
+        imp->putDirect(exec->propertyNames().message, jsString(args[0]->toString(exec)));
+    return obj;
 }
 
 JSValue* NativeErrorImp::callAsFunction(ExecState* exec, JSObject*, const List& args)
 {
-  return construct(exec, args);
+    return construct(exec, args);
 }
 
 void NativeErrorImp::mark()
 {
-  JSObject::mark();
-  if (proto && !proto->marked())
-    proto->mark();
+    JSObject::mark();
+    if (proto && !proto->marked())
+        proto->mark();
 }
+
+} // namespace KJS
index 3e1ac8ae0850013e72368b53a46c95c58485a8fd..cbd80ddc4546ff22d92b6d0a12660ba0f1967e79 100644 (file)
@@ -1,7 +1,6 @@
-// -*- c-basic-offset: 2 -*-
 /*
- *  This file is part of the KDE libraries
  *  Copyright (C) 1999-2000 Harri Porten (porten@kde.org)
+ *  Copyright (C) 2008 Apple Inc. All rights reserved.
  *
  *  This library is free software; you can redistribute it and/or
  *  modify it under the terms of the GNU Lesser General Public
 
 namespace KJS {
 
-  class ErrorInstance : public JSObject {
-  public:
-    ErrorInstance(JSObject *proto);
-    
-    virtual const ClassInfo *classInfo() const { return &info; }
-    static const ClassInfo info;
-  };
-  
-  class ErrorPrototype : public JSObject {
-  public:
-    ErrorPrototype(ExecState *exec,
-                      ObjectPrototype *objectProto,
-                      FunctionPrototype *funcProto);
-  };
-
-  class ErrorProtoFunc : public InternalFunctionImp {
-  public:
-    ErrorProtoFunc(ExecState*, FunctionPrototype*, const Identifier&);
-    virtual JSValue *callAsFunction(ExecState *exec, JSObject *thisObj, const List &args);
-  };
-
-  class ErrorObjectImp : public InternalFunctionImp {
-  public:
-    ErrorObjectImp(ExecState *exec, FunctionPrototype *funcProto,
-                   ErrorPrototype *errorProto);
-
-    virtual bool implementsConstruct() const;
-    virtual JSObject *construct(ExecState *exec, const List &args);
-
-    virtual JSValue *callAsFunction(ExecState *exec, JSObject *thisObj, const List &args);
-  };
-
-  class NativeErrorPrototype : public JSObject {
-  public:
-    NativeErrorPrototype(ExecState *exec, ErrorPrototype *errorProto,
-                            ErrorType et, UString name, UString message);
-  private:
-    ErrorType errType;
-  };
-
-  class NativeErrorImp : public InternalFunctionImp {
-  public:
-    NativeErrorImp(ExecState *exec, FunctionPrototype *funcProto,
-                   JSObject *prot);
-
-    virtual bool implementsConstruct() const;
-    virtual JSObject *construct(ExecState *exec, const List &args);
-    virtual JSValue *callAsFunction(ExecState *exec, JSObject *thisObj, const List &args);
-
-    virtual void mark();
-
-    virtual const ClassInfo *classInfo() const { return &info; }
-    static const ClassInfo info;
-  private:
-    JSObject *proto;
-  };
-
-} // namespace
-
-#endif
+    class ErrorInstance : public JSObject {
+    public:
+        ErrorInstance(JSObject* prototype);
+
+        virtual const ClassInfo* classInfo() const { return &info; }
+        static const ClassInfo info;
+    };
+
+    class ErrorPrototype : public JSObject {
+    public:
+        ErrorPrototype(ExecState*, ObjectPrototype*, FunctionPrototype*);
+    };
+
+    class ErrorProtoFuncToString : public InternalFunctionImp {
+    public:
+        ErrorProtoFuncToString(ExecState*, FunctionPrototype*);
+
+        virtual JSValue* callAsFunction(ExecState*, JSObject*, const List&);
+    };
+
+    class ErrorObjectImp : public InternalFunctionImp {
+    public:
+        ErrorObjectImp(ExecState*, FunctionPrototype*, ErrorPrototype*);
+
+        virtual bool implementsConstruct() const;
+        virtual JSObject* construct(ExecState*, const List&);
+
+        virtual JSValue* callAsFunction(ExecState*, JSObject*, const List&);
+    };
+
+    class NativeErrorPrototype : public JSObject {
+    public:
+        NativeErrorPrototype(ExecState*, ErrorPrototype*, UString name, UString message);
+    };
+
+    class NativeErrorImp : public InternalFunctionImp {
+    public:
+        NativeErrorImp(ExecState*, FunctionPrototype*, JSObject*);
+
+        virtual bool implementsConstruct() const;
+        virtual JSObject* construct(ExecState*, const List&);
+        virtual JSValue* callAsFunction(ExecState*, JSObject*, const List&);
+
+        virtual void mark();
+
+        virtual const ClassInfo* classInfo() const { return &info; }
+        static const ClassInfo info;
+
+    private:
+        JSObject* proto;
+    };
+
+} // namespace KJS
+
+#endif // ERROR_OBJECT_H_