RESOLVED FIXED 15958
base64 spends 1.1% of total time checking for special Infinity case 
https://bugs.webkit.org/show_bug.cgi?id=15958
Summary base64 spends 1.1% of total time checking for special Infinity case
Eric Seidel (no email)
Reported 2007-11-12 12:53:03 PST
base64 spends 1.1% of total time working around toDouble/kjs_dota logic mismatch Well, here's another softball: at least 1.4% of *total time* in base64 is spent on this line in UString::toDouble: if (strncmp(c, "Infinity", 8) == 0) { We already have our own custom dota (aka kjs_dota) we just need a way to call into that logic where it doesn't ever return "Infinity" and instead does what UString::toDouble wants it to.
Attachments
Patch (1.39 KB, patch)
2007-11-12 23:14 PST, Geoffrey Garen 		sam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Geoffrey Garen
Comment 1 2007-11-12 22:25:04 PST
I'll take this.
Geoffrey Garen
Comment 2 2007-11-12 23:14:42 PST
Created attachment 17221 [details] Patch
Sam Weinig
Comment 3 2007-11-12 23:48:12 PST
Comment on attachment 17221 [details] Patch Looking reasonable. r=me.
Eric Seidel (no email)
Comment 4 2007-11-12 23:48:50 PST
Comment on attachment 17221 [details] Patch Oh. I figured we'd had up kjs_dota, but this looks great too. :)
Geoffrey Garen
Comment 5 2007-11-13 00:05:19 PST
Committed revision 27748.
Note You need to log in before you can comment on or make changes to this bug.