commit 6f0849ff831d20953256213f839c398677696f40
author Yi Kong <yikong@google.com> Thu Jul 14 08:03:44 2022 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Thu Jul 14 08:03:44 2022 +0000
tree d3f76efd03c27aa51fc4ed57b99a7c65a7e2da63
parent fddbbb61fea2d6cba6ebf581e13b7a038b326a38
parent e1ed6cb5331e30d54d49f8c6fbbdd31a0f1595f6

Fix performance-inefficient-vector-operation am: e1ed6cb533

Original change: https://android-review.googlesource.com/c/platform/system/apex/+/2152598

Change-Id: I824c331b2b1557ae874624b99bccc7936a944d9a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

apexd/apex_file_repository.cpp

diff --git a/apexd/apex_file_repository.cpp b/apexd/apex_file_repository.cpp
index e82feb4..084f9a7 100644
--- a/apexd/apex_file_repository.cpp
+++ b/apexd/apex_file_repository.cpp
@@ -461,6 +461,7 @@
 
 std::vector<ApexFileRef> ApexFileRepository::GetPreInstalledApexFiles() const {
   std::vector<ApexFileRef> result;
+  result.reserve(pre_installed_store_.size());
   for (const auto& it : pre_installed_store_) {
     result.emplace_back(std::cref(it.second));
   }
@@ -469,6 +470,7 @@
 
 std::vector<ApexFileRef> ApexFileRepository::GetDataApexFiles() const {
   std::vector<ApexFileRef> result;
+  result.reserve(data_store_.size());
   for (const auto& it : data_store_) {
     result.emplace_back(std::cref(it.second));
   }