Changeset 2883 in webkit for trunk/JavaScriptCore/kjs/value.cpp

Timestamp:

Nov 26, 2002, 3:52:00 PM (23 years ago)

Author:

mjs

Message:

Change ActivationImp to be allocated via the garbage collector
again instead of on the stack. This fixes the following four
regressions but sadly it causes a 6% performance hit. It's
probably possibly to reduce the hit a bit by being smarter about
inlining and the way the marking list variant is implemented, but
I'll look into that later.

• fixed 3111500 - REGRESSION: crash in "KJS::ScopeChain::mark()" on www.posci.com
• fixed 3111145 - REGRESSION: reproducible crash in KJS hashtable lookup at time.com
• fixed 3110897 - REGRESSION: javascript crasher on ​http://bmwgallery.tripod.com/
• fixed 3109987 - REGRESSION: Reproducible crash in KJS ObjectImp at live365.com

Also:

• improved DEBUG_COLLECTOR mode a bit by never giving memory back to the system.

• kjs/collector.cpp:
• kjs/context.h:
• kjs/function.cpp: (ActivationImp::ActivationImp): (ActivationImp::mark): (ActivationImp::createArgumentsObject):
• kjs/function.h:
• kjs/internal.cpp: (ContextImp::ContextImp): (ContextImp::mark):
• kjs/list.cpp:
• kjs/list.h:
• kjs/value.cpp: (Value::Value):

File:

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

trunk/JavaScriptCore/kjs/value.cpp

@@ -204 +204 @@
 #if DEBUG_COLLECTOR
   assert (!(rep && !SimpleNumber::is(rep) && *((uint32_t *)rep) == 0 ));
-  assert (!(rep && !SimpleNumber::is(rep) && rep->_flags & VI_MARKED));
+  assert (!(rep && !SimpleNumber::is(rep) && rep->_flags & ValueImp::VI_MARKED));
 #endif
   if (v)
@@ -219 +219 @@
 #if DEBUG_COLLECTOR
   assert (!(rep && !SimpleNumber::is(rep) && *((uint32_t *)rep) == 0 ));
-  assert (!(rep && !SimpleNumber::is(rep) && rep->_flags & VI_MARKED));
+  assert (!(rep && !SimpleNumber::is(rep) && rep->_flags & ValueImp::VI_MARKED));
 #endif
   if (rep)