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

Timestamp:

Aug 9, 2002, 4:24:41 PM (23 years ago)

Author:

mjs

Message:

• fixed 2948835 - JavaScriptCore locking is too fine grained, makes it too slow

• kjs/collector.cpp: (Collector::allocate): (Collector::collect): (Collector::finalCheck): (Collector::numInterpreters): (Collector::numGCNotAllowedObjects): (Collector::numReferencedObjects):
• kjs/collector.h:
• kjs/internal.cpp: (initializeInterpreterLock): (lockInterpreter): (unlockInterpreter): (Parser::parse): (InterpreterImp::InterpreterImp): (InterpreterImp::clear): (InterpreterImp::evaluate):
• kjs/value.cpp: (ValueImp::ValueImp): (ValueImp::setGcAllowed):

File:

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

trunk/JavaScriptCore/kjs/value.cpp

@@ -42 +42 @@
 // ----------------------------- ValueImp -------------------------------------
 
-#if APPLE_CHANGES
-ValueImp::ValueImp() :
-  refcount(0)
-{
-  // Tell the garbage collector that this memory block corresponds to a real object now
-  Collector::lock();
-  _flags = VI_CREATED;
-  //fprintf(stderr,"ValueImp::ValueImp %p\n",(void*)this);
-  Collector::unlock();
-}
-#else
 ValueImp::ValueImp() :
   refcount(0),
@@ -60 +49 @@
   //fprintf(stderr,"ValueImp::ValueImp %p\n",(void*)this);
 }
-#endif
 
 ValueImp::~ValueImp()
@@ -81 +69 @@
 void ValueImp::setGcAllowed()
 {
-#ifdef APPLE_CHANGES
-  Collector::lock();
-#endif
   //fprintf(stderr,"ValueImp::setGcAllowed %p\n",(void*)this);
   _flags |= VI_GCALLOWED;
-#ifdef APPLE_CHANGES
-  Collector::unlock();
-#endif
 }