[webkit-reviews] review requested: [Bug 12987] Fix and import 4XPath test_numeric_expr.html : [Attachment 13491] proposed fix

bugzilla-request-daemon at macosforge.org bugzilla-request-daemon at macosforge.org
Tue Mar 6 11:09:27 PST 2007


Alexey Proskuryakov <ap at webkit.org> has asked  for review:
Bug 12987: Fix and import 4XPath test_numeric_expr.html
http://bugs.webkit.org/show_bug.cgi?id=12987

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

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
Not converting to DeprecatedString in Value::toNumber() helps because
DeprecatedString::ascii() has an apparent bug: all characters above 0xFF are
converted to \0, so toDouble("\u2022abc") returns 0 instead of failure.

I thought about fixing this globally, but the problem is deep inside
DeprecatedStringData:makeAscii(), and changing it now seems too dangerous to
me.



More information about the webkit-reviews mailing list