diff options
| author | Matthias Maennich <matthias@maennich.net> | 2017-09-19 14:16:25 (GMT) |
|---|---|---|
| committer | Brad King <brad.king@kitware.com> | 2017-09-28 11:23:41 (GMT) |
| commit | 79b8c3802a430577a83ead5b0baab7038a813116 (patch) | |
| tree | 0343a05c78438d1b43362c254294e5fb241b2b64 /Source/cmServerProtocol.cxx | |
| parent | a45928cdebcf37de2605e4f58509a37542dd9eba (diff) | |
| download | CMake-79b8c3802a430577a83ead5b0baab7038a813116.zip CMake-79b8c3802a430577a83ead5b0baab7038a813116.tar.gz CMake-79b8c3802a430577a83ead5b0baab7038a813116.tar.bz2 | |
Improve several occurrences of vector::push_back in loops
Fix issues diagnosed by clang-tidy by pre-allocating the vector capacity
before the loop [performance-inefficient-vector-operation].
Signed-off-by: Matthias Maennich <matthias@maennich.net>
Diffstat (limited to 'Source/cmServerProtocol.cxx')
| -rw-r--r-- | Source/cmServerProtocol.cxx | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/Source/cmServerProtocol.cxx b/Source/cmServerProtocol.cxx index 1b47608..e835b7a 100644 --- a/Source/cmServerProtocol.cxx +++ b/Source/cmServerProtocol.cxx @@ -602,11 +602,12 @@ bool LanguageData::operator==(const LanguageData& other) const void LanguageData::SetDefines(const std::set<std::string>& defines) { std::vector<std::string> result; + result.reserve(defines.size()); for (std::string const& i : defines) { result.push_back(i); } std::sort(result.begin(), result.end()); - Defines = result; + Defines = std::move(result); } namespace std { |