WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
146387
Make converting JSString to StringView idiomatically safe
https://bugs.webkit.org/show_bug.cgi?id=146387
Summary
Make converting JSString to StringView idiomatically safe
Darin Adler
Reported
2015-06-27 09:31:19 PDT
Make converting JSString to StringView idiomatically safe
Attachments
Patch
(21.92 KB, patch)
2015-06-27 13:51 PDT
,
Darin Adler
andersca
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Darin Adler
Comment 1
2015-06-27 13:51:13 PDT
Created
attachment 255704
[details]
Patch
Anders Carlsson
Comment 2
2015-06-27 14:46:38 PDT
Comment on
attachment 255704
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=255704&action=review
> Source/JavaScriptCore/runtime/JSString.h:456 > + const JSString* volatile m_string { nullptr };
Might be useful to put a comment here stating why this needs to be volatile.
Darin Adler
Comment 3
2015-06-27 15:54:03 PDT
Committed
r186037
: <
http://trac.webkit.org/changeset/186037
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug