Bug 135351 - Use WTF::move() instead of std::move() to help ensure move semantics
Summary: Use WTF::move() instead of std::move() to help ensure move semantics
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Daniel Bates
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-07-28 13:58 PDT by Daniel Bates
Modified: 2014-07-29 08:48 PDT (History)
9 users (show)

See Also:


Attachments
Patch (8.33 KB, patch)
2014-07-28 14:10 PDT, Daniel Bates
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Bates 2014-07-28 13:58:46 PDT
Substitute WTF::move() for std::move() in code added after the patch for bug #134500.
Comment 1 Daniel Bates 2014-07-28 14:10:01 PDT
Created attachment 235615 [details]
Patch
Comment 2 WebKit Commit Bot 2014-07-28 14:11:54 PDT
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
Comment 3 WebKit Commit Bot 2014-07-28 14:12:07 PDT
Attachment 235615 [details] did not pass style-queue:


ERROR: Source/JavaScriptCore/bytecode/GetByIdVariant.cpp:42:  Wrong number of spaces before statement. (expected: 8)  [whitespace/indent] [4]
Total errors found: 1 in 11 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 Daniel Bates 2014-07-29 08:48:47 PDT
Comment on attachment 235615 [details]
Patch

Clearing flags on attachment: 235615

Committed r171746: <http://trac.webkit.org/changeset/171746>
Comment 5 Daniel Bates 2014-07-29 08:48:51 PDT
All reviewed patches have been landed.  Closing bug.