Title: [103691] trunk/Source/_javascript_Core
- Revision
- 103691
- Author
- [email protected]
- Date
- 2011-12-26 11:55:18 -0800 (Mon, 26 Dec 2011)
Log Message
Fix miss-commit of utf8 change.
Reviewed by Filip Pizlo
Eeep, patch as landed a while ago had no effect! - acidentally landed
modified version of patch used for performance testing.
(This isn't covered by layout tests because layour tests don't use jsc,
and the tests/mozilla tests use latin1, which was already supported!)
Landing changes as intended (and as originally reviewed).
* jsc.cpp:
(jscSource):
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (103690 => 103691)
--- trunk/Source/_javascript_Core/ChangeLog 2011-12-26 19:47:13 UTC (rev 103690)
+++ trunk/Source/_javascript_Core/ChangeLog 2011-12-26 19:55:18 UTC (rev 103691)
@@ -1,3 +1,20 @@
+2011-12-26 Gavin Barraclough <[email protected]>
+
+ Fix miss-commit of utf8 change.
+
+ Reviewed by Filip Pizlo
+
+ Eeep, patch as landed a while ago had no effect! - acidentally landed
+ modified version of patch used for performance testing.
+
+ (This isn't covered by layout tests because layour tests don't use jsc,
+ and the tests/mozilla tests use latin1, which was already supported!)
+
+ Landing changes as intended (and as originally reviewed).
+
+ * jsc.cpp:
+ (jscSource):
+
2011-12-26 Filip Pizlo <[email protected]>
Unreviewed build fix for ARMv7.
Modified: trunk/Source/_javascript_Core/jsc.cpp (103690 => 103691)
--- trunk/Source/_javascript_Core/jsc.cpp 2011-12-26 19:47:13 UTC (rev 103690)
+++ trunk/Source/_javascript_Core/jsc.cpp 2011-12-26 19:55:18 UTC (rev 103691)
@@ -228,7 +228,7 @@
// Slow case - contains non-ascii characters, use fromUTF8WithLatin1Fallback.
ASSERT(*pos < 0);
ASSERT(strlen(utf8) == asciiLength + strlen(pos));
- String source = String(utf8, asciiLength + strlen(pos));
+ String source = String::fromUTF8WithLatin1Fallback(utf8, asciiLength + strlen(pos));
return makeSource(source.impl(), filename);
}
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
