kjs constructor patch

Darin Adler darin at apple.com
Thu Oct 2 12:19:15 CEST 2003


This fixes a problem seen at http://www.moock.org/asdg/codedepot/

Not only do they know about the problem, but their site "moocks" Safari 
because of it! OK, not really.

Index: kjs/nodes.cpp
===================================================================
RCS file: /local/home/cvs/Labyrinth/JavaScriptCore/kjs/nodes.cpp,v
retrieving revision 1.41
diff -p -u -u -p -r1.41 kjs/nodes.cpp
--- kjs/nodes.cpp	2003/05/15 20:47:10	1.41
+++ kjs/nodes.cpp	2003/10/02 18:15:10
@@ -2798,7 +2798,8 @@ void FuncDeclNode::processFuncDecl(ExecS

    //  Value proto = 
exec->interpreter()->builtinObject().construct(exec,List::empty());
    List empty;
-  Value proto = 
exec->interpreter()->builtinObject().construct(exec,empty);
+  Object proto = 
exec->interpreter()->builtinObject().construct(exec,empty);
+  proto.put(exec, constructorPropertyName, func, 
ReadOnly|DontDelete|DontEnum);
    func.put(exec, prototypePropertyName, proto, Internal|DontDelete);

    int plen = 0;
===================================================================

     -- Darin



More information about the Khtml-devel mailing list