[webkit-changes] cvs commit: JavaScriptCore/kjs property_map.cpp

Darin darin at opensource.apple.com
Sun Aug 14 19:55:14 PDT 2005


darin       05/08/14 19:55:14

  Modified:    .        ChangeLog
               kjs      property_map.cpp
  Log:
          Reviewed by Maciej.
  
          - fixed crash observed on one of the Apple-only layout tests
  
          * kjs/property_map.cpp: (KJS::PropertyMap::mark): Change code to understand that deleted
          entries have a value of NULL, so the deleted sentinel count doesn't need to be included
          in the count of things to mark since we're ignoring the keys.
  
  Revision  Changes    Path
  1.797     +10 -0     JavaScriptCore/ChangeLog
  
  Index: ChangeLog
  ===================================================================
  RCS file: /cvs/root/JavaScriptCore/ChangeLog,v
  retrieving revision 1.796
  retrieving revision 1.797
  diff -u -r1.796 -r1.797
  --- ChangeLog	14 Aug 2005 16:41:47 -0000	1.796
  +++ ChangeLog	15 Aug 2005 02:55:13 -0000	1.797
  @@ -2,6 +2,16 @@
   
           Reviewed by Maciej.
   
  +        - fixed crash observed on one of the Apple-only layout tests
  +
  +        * kjs/property_map.cpp: (KJS::PropertyMap::mark): Change code to understand that deleted
  +        entries have a value of NULL, so the deleted sentinel count doesn't need to be included
  +        in the count of things to mark since we're ignoring the keys.
  +
  +2005-08-14  Darin Adler  <darin at apple.com>
  +
  +        Reviewed by Maciej.
  +
           - fixed http://bugzilla.opendarwin.org/show_bug.cgi?id=4421
             speed up JavaScript by inlining some label stack functions
   
  
  
  
  1.47      +1 -1      JavaScriptCore/kjs/property_map.cpp
  
  Index: property_map.cpp
  ===================================================================
  RCS file: /cvs/root/JavaScriptCore/kjs/property_map.cpp,v
  retrieving revision 1.46
  retrieving revision 1.47
  diff -u -r1.46 -r1.47
  --- property_map.cpp	14 Aug 2005 16:27:13 -0000	1.46
  +++ property_map.cpp	15 Aug 2005 02:55:14 -0000	1.47
  @@ -541,7 +541,7 @@
           return;
       }
   
  -    int minimumKeysToProcess = _table->keyCount + _table->sentinelCount;
  +    int minimumKeysToProcess = _table->keyCount;
       Entry *entries = _table->entries;
       for (int i = 0; i < minimumKeysToProcess; i++) {
           ValueImp *v = entries[i].value;
  
  
  



More information about the webkit-changes mailing list