RESOLVED FIXED176302
String#utf8() allocates new CString 
https://bugs.webkit.org/show_bug.cgi?id=176302
Summary String#utf8() allocates new CString
Yusuke Suzuki
Reported 2017-09-03 06:05:06 PDT
String#utf8() allocates new CString
Attachments
Patch (5.06 KB, patch)
2017-09-03 06:08 PDT, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1 2017-09-03 06:08:45 PDT
Created attachment 319774 [details] Patch
Yusuke Suzuki
Comment 2 2017-09-03 06:16:41 PDT
This issue makes me consider renaming ::utf8() more descriptive one. Like, convertToUTF8()...
Yusuke Suzuki
Comment 3 2017-09-03 18:08:46 PDT
Comment on attachment 319774 [details] Patch Thx!
WebKit Commit Bot
Comment 4 2017-09-03 18:38:13 PDT
Comment on attachment 319774 [details] Patch Clearing flags on attachment: 319774 Committed r221562: <http://trac.webkit.org/changeset/221562>
WebKit Commit Bot
Comment 5 2017-09-03 18:38:14 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 6 2017-09-27 12:42:20 PDT
<rdar://problem/34693796>
Note You need to log in before you can comment on or make changes to this bug.