Todd Lipcon has submitted this change and it was merged. Change subject: Support move constructor and assignment operator in scoped_refptr ......................................................................
Support move constructor and assignment operator in scoped_refptr This is a port over of 4c3de21a3dcdd9e27a770572350eeaa98cdb9e87 from the chromium-base repository. The port isn't 100% -- on their side they added a .Pass() to scoped_refptr. However, I think in our case we should just use std::move() for this use case. Change-Id: I339c9f6c3bfc0fcacd44aa53bdd75d723b9aab47 Reviewed-on: http://gerrit.cloudera.org:8080/1870 Tested-by: Todd Lipcon <[email protected]> Reviewed-by: Dan Burkert <[email protected]> --- M src/kudu/gutil/ref_counted.h 1 file changed, 19 insertions(+), 0 deletions(-) Approvals: Dan Burkert: Looks good to me, approved Todd Lipcon: Verified -- To view, visit http://gerrit.cloudera.org:8080/1870 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I339c9f6c3bfc0fcacd44aa53bdd75d723b9aab47 Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
