diff options
author | 2024-05-08 14:46:36 +0000 | |
---|---|---|
committer | 2024-05-15 10:09:42 +0000 | |
commit | ae7e228b10cd071307445497ce2a158b20b0a2aa (patch) | |
tree | 85211974dff453671a146176040c782f56405b88 /artd/file_utils.cc | |
parent | 958c7e16a66c1dab9e6742602de8d4a4519aa738 (diff) |
Fix performance-inefficient-vector-operation clang-tidy issues
Bug: 264654008
Test: m tidy-art
Change-Id: I52338492e929421758ba924de8708102a71c8ac2
Diffstat (limited to 'artd/file_utils.cc')
-rw-r--r-- | artd/file_utils.cc | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/artd/file_utils.cc b/artd/file_utils.cc index 195daf6162..c1977fb1bb 100644 --- a/artd/file_utils.cc +++ b/artd/file_utils.cc @@ -234,10 +234,13 @@ android::base::Result<void> MoveAllOrAbandon( const std::vector<std::pair<std::string, std::string>>& files_to_move, const std::vector<std::string>& files_to_remove) { std::vector<std::pair<std::string_view, std::string_view>> files_to_move_view; - std::vector<std::string_view> files_to_remove_view; + files_to_move_view.reserve(files_to_move.size()); for (const auto& [src, dst] : files_to_move) { files_to_move_view.emplace_back(src, dst); } + + std::vector<std::string_view> files_to_remove_view; + files_to_remove_view.reserve(files_to_remove.size()); for (const std::string& file : files_to_remove) { files_to_remove_view.emplace_back(file); } |