Title: [130413] trunk/Source/_javascript_Core
Revision
130413
Author
msab...@apple.com
Date
2012-10-04 11:49:42 -0700 (Thu, 04 Oct 2012)

Log Message

After r130344, OpaqueJSString() creates a empty string which should be a null string
https://bugs.webkit.org/show_bug.cgi?id=98417

Reviewed by Alexey Proskuryakov.

Removed the setting of enclosed string to an empty string from default constructor.
Before changeset r130344, the semantic was the default constructor produced a null
string.

* API/OpaqueJSString.h:
(OpaqueJSString::OpaqueJSString):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/API/OpaqueJSString.h (130412 => 130413)


--- trunk/Source/_javascript_Core/API/OpaqueJSString.h	2012-10-04 18:40:56 UTC (rev 130412)
+++ trunk/Source/_javascript_Core/API/OpaqueJSString.h	2012-10-04 18:49:42 UTC (rev 130413)
@@ -59,7 +59,6 @@
 
     OpaqueJSString()
     {
-        m_string = emptyString();
     }
 
     OpaqueJSString(const String& string)

Modified: trunk/Source/_javascript_Core/ChangeLog (130412 => 130413)


--- trunk/Source/_javascript_Core/ChangeLog	2012-10-04 18:40:56 UTC (rev 130412)
+++ trunk/Source/_javascript_Core/ChangeLog	2012-10-04 18:49:42 UTC (rev 130413)
@@ -1,3 +1,17 @@
+2012-10-04  Michael Saboff  <msab...@apple.com>
+
+        After r130344, OpaqueJSString() creates a empty string which should be a null string
+        https://bugs.webkit.org/show_bug.cgi?id=98417
+
+        Reviewed by Alexey Proskuryakov.
+
+        Removed the setting of enclosed string to an empty string from default constructor.
+        Before changeset r130344, the semantic was the default constructor produced a null
+        string.
+
+        * API/OpaqueJSString.h:
+        (OpaqueJSString::OpaqueJSString):
+
 2012-10-04  Csaba Osztrogonác  <o...@webkit.org>
 
         [Qt] Add missing LLInt dependencies to the build system
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to