Changeset 51307 in webkit for trunk/JavaScriptCore/runtime

Timestamp:

Nov 23, 2009, 5:02:13 AM (16 years ago)

Author:

Simon Hausmann

Message:

[Symbian] Fix lastIndexOf() for Symbian
​https://bugs.webkit.org/show_bug.cgi?id=31773

Patch by Laszlo Gombos <Laszlo Gombos> on 2009-11-23
Reviewed by Kenneth Rohde Christiansen.

Symbian soft floating point library has problems with operators
comparing NaN to numbers. Without a workaround lastIndexOf()
function does not work.

Patch developed by David Leong.

• runtime/StringPrototype.cpp:

(JSC::stringProtoFuncLastIndexOf):Add an extra test
to check for NaN for Symbian.

File:

• trunk/JavaScriptCore/runtime/StringPrototype.cpp (modified) (1 diff)

trunk/JavaScriptCore/runtime/StringPrototype.cpp

@@ -470 +470 @@
     else if (!(dpos <= len)) // true for NaN
         dpos = len;
+#if PLATFORM(SYMBIAN)
+    // Work around for broken NaN compare operator
+    else if (isnan(dpos))
+        dpos = len;
+#endif
     return jsNumber(exec, s.rfind(u2, static_cast<int>(dpos)));
 }