From f480380df5107c2204ebbd250282dec18cc2a20d Mon Sep 17 00:00:00 2001 From: Adam Midvidy Date: Mon, 24 Aug 2015 19:01:49 -0400 Subject: SERVER-18668 remove some pessimizing and redundant moves --- src/mongo/rpc/command_reply.cpp | 4 ++-- src/mongo/rpc/command_request.cpp | 4 ++-- src/mongo/rpc/document_range.cpp | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) (limited to 'src/mongo/rpc') diff --git a/src/mongo/rpc/command_reply.cpp b/src/mongo/rpc/command_reply.cpp index 954d6b051d6..b2472230e52 100644 --- a/src/mongo/rpc/command_reply.cpp +++ b/src/mongo/rpc/command_reply.cpp @@ -52,8 +52,8 @@ CommandReply::CommandReply(const Message* message) : _message(message) { const char* messageEnd = begin + length; ConstDataRangeCursor cur(begin, messageEnd); - _metadata = std::move(uassertStatusOK(cur.readAndAdvance>()).val); - _commandReply = std::move(uassertStatusOK(cur.readAndAdvance>()).val); + _metadata = uassertStatusOK(cur.readAndAdvance>()).val; + _commandReply = uassertStatusOK(cur.readAndAdvance>()).val; _outputDocs = DocumentRange(cur.data(), messageEnd); } diff --git a/src/mongo/rpc/command_request.cpp b/src/mongo/rpc/command_request.cpp index 961c4ba7d93..aec10f6fcd7 100644 --- a/src/mongo/rpc/command_request.cpp +++ b/src/mongo/rpc/command_request.cpp @@ -89,8 +89,8 @@ CommandRequest::CommandRequest(const Message* message) : _message(message) { (_commandName.size() >= kMinCommandNameLength) && (_commandName.size() <= kMaxCommandNameLength)); - _metadata = std::move(uassertStatusOK(cur.readAndAdvance>()).val); - _commandArgs = std::move(uassertStatusOK(cur.readAndAdvance>()).val); + _metadata = uassertStatusOK(cur.readAndAdvance>()).val; + _commandArgs = uassertStatusOK(cur.readAndAdvance>()).val; _inputDocs = DocumentRange{cur.data(), messageEnd}; } diff --git a/src/mongo/rpc/document_range.cpp b/src/mongo/rpc/document_range.cpp index aff2cf6dfec..fd525a5e244 100644 --- a/src/mongo/rpc/document_range.cpp +++ b/src/mongo/rpc/document_range.cpp @@ -83,7 +83,7 @@ DocumentRange::const_iterator& DocumentRange::const_iterator::operator++() { if (_cursor.length() == 0) { *this = const_iterator{}; } else { - _obj = std::move(uassertStatusOK(_cursor.readAndAdvance>()).val); + _obj = uassertStatusOK(_cursor.readAndAdvance>()).val; } return *this; } -- cgit v1.2.1