Summary: | [WTF] Fix clang tidy bugprone-move-forwarding-reference static analyzer warnings in CompletionHandler.h | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | David Kilzer (:ddkilzer) <ddkilzer> | ||||||
Component: | Web Template Framework | Assignee: | David Kilzer (:ddkilzer) <ddkilzer> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | benjamin, cdumez, cmarcelo, darin, ews-watchlist, webkit-bug-importer, ysuzuki | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | Other | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
See Also: |
https://bugs.webkit.org/show_bug.cgi?id=229114 https://bugs.webkit.org/show_bug.cgi?id=236202 https://bugs.webkit.org/show_bug.cgi?id=236459 |
||||||||
Attachments: |
|
Description
David Kilzer (:ddkilzer)
2022-02-05 12:39:09 PST
Created attachment 450995 [details]
Patch v1
Created attachment 450996 [details]
Patch v2
(In reply to David Kilzer (:ddkilzer) from comment #2) > Created attachment 450995 [details] > Patch v1 Oops. Changed std::forward() to std::forward<CallableType>(). Comment on attachment 450996 [details]
Patch v2
Marking xq+ since enough builds and tests have run to show this didn't cause a regression.
Committed r289162 (246858@main): <https://commits.webkit.org/246858@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 450996 [details]. Comment on attachment 450996 [details]
Patch v2
r=me, yes, it can be a lref or rref (b/c CallableType is template type of this function), thus we should use std::forward.
Ah, oops it was already landed. |