[webkit-reviews] review granted: [Bug 23944] KURLGoogle's pre-parsed canonicalSpec constructor should take a CString : [Attachment 27643] v1 patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Feb 12 23:50:09 PST 2009


Eric Seidel <eric at webkit.org> has granted Darin Fisher (:fishd, Google)
<darin at chromium.org>'s request for review:
Bug 23944: KURLGoogle's pre-parsed canonicalSpec constructor should take a
CString
https://bugs.webkit.org/show_bug.cgi?id=23944

Attachment 27643: v1 patch
https://bugs.webkit.org/attachment.cgi?id=27643&action=review

------- Additional Comments from Eric Seidel <eric at webkit.org>
Looks fine to me.  I much prefer CString over char* + len anyway!


More information about the webkit-reviews mailing list