Summary: | Make StringView capable of being passed /returned in only 2 registers. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Mark Lam <mark.lam> | ||||
Component: | Web Template Framework | Assignee: | Mark Lam <mark.lam> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | andersca, ap, benjamin, cdumez, cmarcelo, commit-queue, darin, ddkilzer, fpizlo, ggaren, keith_miller, msaboff, saam, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Local Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | 160350 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Mark Lam
2016-07-29 10:12:52 PDT
Created attachment 284872 [details]
proposed patch.
Comment on attachment 284872 [details]
proposed patch.
/Volumes/Data/EWS/WebKit/Source/WebCore/rendering/svg/SVGInlineTextBox.cpp:615:13: error: unused variable 'selectionTextRun' [-Werror,-Wunused-variable]
TextRun selectionTextRun = constructTextRun(selectionStyle, fragment);
(In reply to comment #2) > Comment on attachment 284872 [details] > proposed patch. > > /Volumes/Data/EWS/WebKit/Source/WebCore/rendering/svg/SVGInlineTextBox.cpp: > 615:13: error: unused variable 'selectionTextRun' [-Werror,-Wunused-variable] > TextRun selectionTextRun = constructTextRun(selectionStyle, fragment); Does not appear to be related to my patch at all. Comment on attachment 284872 [details]
proposed patch.
r=me
Comment on attachment 284872 [details]
proposed patch.
I'll land this manually later.
Thanks for the review. Landed in r203911: <http://trac.webkit.org/r203911>. Thanks, Mark. This is great! Follow up build fix in r203915: <http://trac.webkit.org/r203915>. > #if defined(NDEBUG) || COMPILER(MSVC) || 1
> #define CHECK_STRINGVIEW_LIFETIME 0
This is pre-existing code, however I'm curious anyway - is there a bug tracking re-enabling the checks?
Wow, did not realize these were turned off. Do we know exactly when we turned them off and why? Should find out when that || 1 was added. (In reply to comment #11) > Wow, did not realize these were turned off. Do we know exactly when we > turned them off and why? Should find out when that || 1 was added. See https://trac.webkit.org/changeset/174397 from Oct 2014. (In reply to comment #10) > This is pre-existing code, however I'm curious anyway - is there a bug > tracking re-enabling the checks? Apparently, no. Will work on re-enabling the StringView life-cycle checks in https://bugs.webkit.org/show_bug.cgi?id=160384. |