2007-11-06 Eric Seidel <eric@webkit.org>
authoreseidel <eseidel@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Nov 2007 01:03:26 +0000 (01:03 +0000)
committereseidel <eseidel@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Nov 2007 01:03:26 +0000 (01:03 +0000)
        Reviewed by darin.

        This fixes a regressed layout test for string + object

        SunSpider claims this was an overall 0.3% speedup, although some individual tests were slower.

        * kjs/nodes.cpp:
        (KJS::add): remove erroneous "fast path" for string + *

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@27494 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JavaScriptCore/ChangeLog
JavaScriptCore/kjs/nodes.cpp

index 4f36019ac6ce831d4bf40be4dd8f00267d154445..f00b09408cdace8c245bc4c3b19a9e2f3298f853 100644 (file)
@@ -1,3 +1,14 @@
+2007-11-06  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by darin.
+
+        This fixes a regressed layout test for string + object
+        
+        SunSpider claims this was an overall 0.3% speedup, although some individual tests were slower.
+
+        * kjs/nodes.cpp:
+        (KJS::add): remove erroneous "fast path" for string + *
+
 2007-11-06  Geoffrey Garen  <ggaren@apple.com>
 
         Reviewed by Eric Seidel.
index e236b4fc283416785e638bf98abec8c7145bdbd9..70799e383e8e7761314c25af72603f7881ea3c87 100644 (file)
@@ -1701,11 +1701,6 @@ static inline JSValue* add(ExecState* exec, JSValue* v1, JSValue *v2)
         if (value.isNull())
             return throwOutOfMemoryError(exec);
         return jsString(value);
-    } else if (t1 == StringType) { // common js idiom "" + object
-        UString value = static_cast<StringImp*>(v1)->value() + v2->toString(exec);
-        if (value.isNull())
-            return throwOutOfMemoryError(exec);
-        return jsString(value);
     }
     
     // All other cases are pretty uncommon