Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20484 )
Change subject: [misc] replace vector::resize() with constructor's parameter ...................................................................... [misc] replace vector::resize() with constructor's parameter I noticed the following pattern that seems to be replicated in various places: std::vector<T> v; v.resize(some_constant); This patch replaces such code with calls to one of std::vector's constructors that accepts the size of the vectors as a parameter. I also took the liberty of updating the related code to be more style-compliant and making other minor improvements. This patch doesn't contain any functional modifications. Change-Id: Id72689597c39b66a6f2f24f734a12a90afef152a Reviewed-on: http://gerrit.cloudera.org:8080/20484 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/benchmarks/wal_hiccup.cc M src/kudu/cfile/encoding-test.cc M src/kudu/client/master_rpc.cc M src/kudu/client/schema.cc M src/kudu/common/partition.cc M src/kudu/common/rowblock.cc M src/kudu/experiments/merge-test.cc M src/kudu/fs/log_block_manager.cc M src/kudu/master/txn_manager-test.cc M src/kudu/rpc/rpc_stub-test.cc M src/kudu/rpc/server_negotiation.cc M src/kudu/security/tls_handshake-test.cc M src/kudu/tablet/compaction.cc M src/kudu/tablet/delta_compaction.cc M src/kudu/tablet/deltafile-test.cc M src/kudu/tablet/deltamemstore-test.cc M src/kudu/tablet/lock_manager.cc M src/kudu/tablet/rowset_tree.cc M src/kudu/tablet/tablet-test-base.h M src/kudu/tablet/tablet_random_access-test.cc M src/kudu/tserver/tablet_copy_source_session.cc M src/kudu/util/maintenance_manager.cc M src/kudu/util/rle-test.cc M src/kudu/util/threadpool-test.cc M src/kudu/util/url-coding-test.cc 25 files changed, 64 insertions(+), 79 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/20484 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id72689597c39b66a6f2f24f734a12a90afef152a Gerrit-Change-Number: 20484 Gerrit-PatchSet: 4 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
