[webkit-reviews] review denied: [Bug 15718] ASSERTION FAILED: _hash in KJS::UString::Rep::computedHash() : [Attachment 16902] fix 2

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has denied 's request for review:
Bug 15718: ASSERTION FAILED: _hash in KJS::UString::Rep::computedHash()
http://bugs.webkit.org/show_bug.cgi?id=15718

Attachment 16902: fix 2
http://bugs.webkit.org/attachment.cgi?id=16902&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
+    if (!strlen(c)) {

Need the O(1) check of c[0] instead of the O(string-length) check of strlen.

I think it would be even better to do this in UString -- guarantee that null
and empty both have a precomputed hash. It's annoying to have extra overhead in
Identifier::add for 1-time setup.