Bug 176302

Summary: String#utf8() allocates new CString
Product: WebKit Reporter: Yusuke Suzuki <ysuzuki>
Component: New BugsAssignee: Yusuke Suzuki <ysuzuki>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, commit-queue, sam, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

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.