Bug 137809

Summary: Use WTF::move() instead of std::move() to help ensure move semantics in JavaScriptCore
Product: WebKit Reporter: Gyuyoung Kim <gyuyoung.kim>
Component: JavaScriptCoreAssignee: Gyuyoung Kim <gyuyoung.kim>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, ossy
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 128007    
Attachments:
Description Flags
Patch none

Gyuyoung Kim
Reported 2014-10-17 00:11:26 PDT
There are still some use of std::move() in JavaScriptCore.
Attachments
Patch (3.06 KB, patch)
2014-10-17 00:12 PDT, Gyuyoung Kim 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Gyuyoung Kim
Comment 1 2014-10-17 00:12:50 PDT
Created attachment 240002 [details] Patch
Csaba Osztrogonác
Comment 2 2014-10-17 02:18:41 PDT
Comment on attachment 240002 [details] Patch r=me It seems they are from the ftlopt branch merge, I assume we didn't have style checker for this that time.
WebKit Commit Bot
Comment 3 2014-10-17 02:54:14 PDT
Comment on attachment 240002 [details] Patch Clearing flags on attachment: 240002 Committed r174815: <http://trac.webkit.org/changeset/174815>
WebKit Commit Bot
Comment 4 2014-10-17 02:54:17 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.