Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15583 )
Change subject: gutil: remove callback and bind from the codebase ...................................................................... gutil: remove callback and bind from the codebase It is fitting that 5.5 years after bringing this code into Kudu (see commit 31f072096), I'm lucky enough to be the one burying it. There are no more usages of this functionality; everything has been migrated to std::function with lambdas for binding/capturing. Change-Id: Ib5fb53730f9c33b37415b22fcc775266467116b9 Reviewed-on: http://gerrit.cloudera.org:8080/15583 Tested-by: Kudu Jenkins Reviewed-by: Andrew Wong <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M build-support/release/rat_exclude_files.txt M docs/contributing.adoc M src/kudu/gutil/CMakeLists.txt D src/kudu/gutil/bind.h D src/kudu/gutil/bind.h.pump D src/kudu/gutil/bind_helpers.h D src/kudu/gutil/bind_internal.h D src/kudu/gutil/bind_internal.h.pump D src/kudu/gutil/callback.h D src/kudu/gutil/callback.h.pump D src/kudu/gutil/callback_forward.h D src/kudu/gutil/callback_internal.cc D src/kudu/gutil/callback_internal.h D src/kudu/gutil/raw_scoped_refptr_mismatch_checker.h M src/kudu/util/CMakeLists.txt D src/kudu/util/callback_bind-test.cc 16 files changed, 11 insertions(+), 6,046 deletions(-) Approvals: Kudu Jenkins: Verified Andrew Wong: Looks good to me, approved Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/15583 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib5fb53730f9c33b37415b22fcc775266467116b9 Gerrit-Change-Number: 15583 Gerrit-PatchSet: 6 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
