Bug 135351

Summary:

Use WTF::move() instead of std::move() to help ensure move semantics

Product:

WebKit

Reporter:

Daniel Bates <dbates>

Component:

WebCore Misc.

Assignee:

Daniel Bates <dbates>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

andersca, benjamin, berto, cgarcia, cmarcelo, commit-queue, gustavo, mrobinson, sam

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Patch	none

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.