Bug 230282

Summary: Fix -Wredundant-move warnings
Product: WebKit Reporter: Michael Catanzaro <mcatanzaro>
Component: WebKit2Assignee: Michael Catanzaro <mcatanzaro>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, kkinnunen, mcatanzaro, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Patch none

Description Michael Catanzaro 2021-09-14 15:59:25 PDT
Fix -Wredundant-move warnings

Hi Alex, I'm CCing you because you introduced one of these in 241547@main, and it technically requires owner approval to fix....
Comment 1 Michael Catanzaro 2021-09-14 16:00:28 PDT
Created attachment 438185 [details]
Patch
Comment 2 Michael Catanzaro 2021-09-14 16:01:47 PDT
(BTW the difference here is: GCC warns whenever a WTFMove is redundant in return values, whereas clang only warns when a WTFMove actually prevents copy elision from happening. I don't know the differences between the two cases.)
Comment 3 Michael Catanzaro 2021-09-14 16:04:13 PDT
Comment on attachment 438185 [details]
Patch

Thanks.
Comment 4 EWS 2021-09-14 16:48:01 PDT
Committed r282421 (241680@main): <https://commits.webkit.org/241680@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 438185 [details].
Comment 5 Radar WebKit Bug Importer 2021-09-14 16:48:22 PDT
<rdar://problem/83122939>