[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