30 Jun
2009
30 Jun
'09
4:57 p.m.
I have noticed that rather than using memcmp for comparing strings, webkit likes to cast to uint32_t* and compare in a for loop. For example, WebCore::equal in AtomicString.cpp and StringHash::equal in StringHash.h. Is there any reason not to memcmp? I am assuming that most implementations of memcmp will do the word alignment and basically do the same thing. The reason I ask is because StringHash::equal does not check for PLATFORM(ARM) or PLATFORM(SH4) so I was going to submit a patch to use memcmp for ARM/SH4 but didn't know if that was frowned upon (and WebCore::equal does a loop for ARM/SH4 as well). Thanks, Patrick