Bug 176302

Summary:

String#utf8() allocates new CString

Product:

WebKit

Reporter:

Yusuke Suzuki <ysuzuki>

Component:

New Bugs

Assignee:

Yusuke Suzuki <ysuzuki>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

achristensen, commit-queue, sam, webkit-bug-importer, youennf

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none

Description Yusuke Suzuki 2017-09-03 06:05:06 PDT

String#utf8() allocates new CString

Comment 1 Yusuke Suzuki 2017-09-03 06:08:45 PDT

Created attachment 319774 [details]
Patch

Comment 2 Yusuke Suzuki 2017-09-03 06:16:41 PDT

This issue makes me consider renaming ::utf8() more descriptive one. Like, convertToUTF8()...

Comment 3 Yusuke Suzuki 2017-09-03 18:08:46 PDT

Comment on attachment 319774 [details]
Patch

Thx!

Comment 4 WebKit Commit Bot 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>

Comment 5 WebKit Commit Bot 2017-09-03 18:38:14 PDT

All reviewed patches have been landed.  Closing bug.

Comment 6 Radar WebKit Bug Importer 2017-09-27 12:42:20 PDT

<rdar://problem/34693796>