clang-tidy: Enable performance-inefficient-vector-operation check
Finds possible inefficient std::vector operations (e.g. push_back,
emplace_back) that may cause unnecessary memory reallocations.
It can also find calls that add element to protobuf repeated field in
a loop without calling Reserve() before the loop. Calling Reserve()
first can avoid unnecessary memory reallocations.
Signed-off-by: George Liu <liuxiwei@ieisystem.com>
Change-Id: I442c3aac7e3c71abbb125ae6bcdbe3f8995d14e9
diff --git a/src/utils.cpp b/src/utils.cpp
index 9d32f3b..3b611d7 100644
--- a/src/utils.cpp
+++ b/src/utils.cpp
@@ -113,6 +113,7 @@
throw std::runtime_error("Error reading mapper response");
}
std::vector<std::string> ret;
+ ret.reserve(mapperResponse.size());
for (const auto& i : mapperResponse)
{
ret.emplace_back(i.first);