Todd Lipcon has submitted this change and it was merged. Change subject: Support move construction and assignment for Status ......................................................................
Support move construction and assignment for Status This ought to avoid some extra allocations and copies in the common case of returning Status from a function. The new support is enabled only in C++11, since status.h is also sometimes included by non-C++11 client users. Change-Id: Id91aff98764d6df4857c0123687dc417c3d97281 Reviewed-on: http://gerrit.cloudera.org:8080/1871 Reviewed-by: Dan Burkert <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/util/status-test.cc M src/kudu/util/status.h 2 files changed, 71 insertions(+), 0 deletions(-) Approvals: Dan Burkert: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/1871 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id91aff98764d6df4857c0123687dc417c3d97281 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
