Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16261 )
Change subject: Remove last vestiges of gscoped_ptr ...................................................................... Remove last vestiges of gscoped_ptr Adar did most of the work here a few months back. This changes the last usages of gscoped_array over to unique_ptr<T[]> and removes gscoped_ptr entirely from the code base. May it rest in peace. Change-Id: I03832c310b5804227abf35f58aa320e5e81151f6 Reviewed-on: http://gerrit.cloudera.org:8080/16261 Reviewed-by: Adar Lieber-Dembo <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Kudu Jenkins --- M build-support/release/rat_exclude_files.txt M docs/contributing.adoc D src/kudu/gutil/gscoped_ptr.h M src/kudu/gutil/stl_util.h M src/kudu/gutil/strings/escaping.cc M src/kudu/gutil/strings/join.cc M src/kudu/gutil/strings/numbers.cc 7 files changed, 13 insertions(+), 873 deletions(-) Approvals: Adar Lieber-Dembo: Looks good to me, approved Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/16261 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I03832c310b5804227abf35f58aa320e5e81151f6 Gerrit-Change-Number: 16261 Gerrit-PatchSet: 2 Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Lieber-Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
