Hello Adar Lieber-Dembo, Grant Henke,

I'd like you to do a code review. Please visit

    http://gerrit.cloudera.org:8080/16261

to review the following change.


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
---
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(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/61/16261/1
--
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: newchange
Gerrit-Change-Id: I03832c310b5804227abf35f58aa320e5e81151f6
Gerrit-Change-Number: 16261
Gerrit-PatchSet: 1
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Lieber-Dembo <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>

Reply via email to