Changeset 1881 in webkit for trunk/JavaScriptCore

Timestamp:

Aug 20, 2002, 2:53:41 PM (23 years ago)

Author:

mjs

Message:

• kjs/object.cpp: Don't reference other ValueImps in the destructor, they may ha ve already been destroyed, and will have GC_ALLOWED set already in any case.

Location:

trunk/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• ChangeLog-2002-12-03 (modified) (1 diff)
• ChangeLog-2003-10-25 (modified) (1 diff)
• kjs/object.cpp (modified) (1 diff)

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2002-08-20  Maciej Stachowiak  <[email protected]>
+
+        * kjs/object.cpp: Don't reference other ValueImps in the
+        destructor, they may ha ve already been destroyed, and will have
+        GC_ALLOWED set already in any case.
+w
 2002-08-19  Maciej Stachowiak  <[email protected]>
 

trunk/JavaScriptCore/ChangeLog-2002-12-03

@@ +1 @@
+2002-08-20  Maciej Stachowiak  <[email protected]>
+
+        * kjs/object.cpp: Don't reference other ValueImps in the
+        destructor, they may ha ve already been destroyed, and will have
+        GC_ALLOWED set already in any case.
+w
 2002-08-19  Maciej Stachowiak  <[email protected]>
 

trunk/JavaScriptCore/ChangeLog-2003-10-25

@@ +1 @@
+2002-08-20  Maciej Stachowiak  <[email protected]>
+
+        * kjs/object.cpp: Don't reference other ValueImps in the
+        destructor, they may ha ve already been destroyed, and will have
+        GC_ALLOWED set already in any case.
+w
 2002-08-19  Maciej Stachowiak  <[email protected]>
 

trunk/JavaScriptCore/kjs/object.cpp

@@ -80 +80 @@
 {
   //fprintf(stderr,"ObjectImp::~ObjectImp %p\n",(void*)this);
-  if (_proto)
-    _proto->setGcAllowed();
-  if (_internalValue)
-    _internalValue->setGcAllowed();
-  if (_scope)
-    _scope->setGcAllowed();
   delete _prop;
 }