RESOLVED FIXED 6752
Make pointer hash the default for all pointers (plus other Hash improvements) 
https://bugs.webkit.org/show_bug.cgi?id=6752
Summary Make pointer hash the default for all pointers (plus other Hash improvements)
Maciej Stachowiak
Reported 2006-01-23 22:27:30 PST
- renamed PointerHash to PtrHash - made PtrHash the default hash function for int and pointer types that aren't further specialized - added an AtomicStringImpl class to make it easier and more typesafe to identity hash atomic strings - did appropriate consequent cleanup (very few places now need to declare a hash function)
Attachments
cleanup patch (66.46 KB, patch)
2006-01-23 22:33 PST, Maciej Stachowiak 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Maciej Stachowiak
Comment 1 2006-01-23 22:33:35 PST
Created attachment 5902 [details] cleanup patch
Darin Adler
Comment 2 2006-01-23 23:27:29 PST
Comment on attachment 5902 [details] cleanup patch Funny, I have the same patch for htmlfactory.cpp -- I can discard that now. + AtomicString n = getAttribute(nameAttr); Why not make that const AtomicString&? Also, HTMLGenericFormElementImpl::name() can also return const AtomicString& if we have a way to get a reference to a global empty atomic string. In general it's great to have functions that return const AtomicString& when they are attributes to avoid any reference count churn. Looks good. I didn't spot any problems. r=me
Note You need to log in before you can comment on or make changes to this bug.