diff options
| author | Matthias Maennich <matthias@maennich.net> | 2017-09-19 13:54:08 (GMT) |
|---|---|---|
| committer | Brad King <brad.king@kitware.com> | 2017-09-28 11:23:40 (GMT) |
| commit | bb0ad1bea81f60cbbb2c66c2c0ab24ee12db0026 (patch) | |
| tree | 06db4eb05a227219eef3e265fef60f601640d051 /Source/cmGlobalXCodeGenerator.cxx | |
| parent | 8cc33aeaec2798c9ff5a3af23e361ee22fa2ba95 (diff) | |
| download | CMake-bb0ad1bea81f60cbbb2c66c2c0ab24ee12db0026.zip CMake-bb0ad1bea81f60cbbb2c66c2c0ab24ee12db0026.tar.gz CMake-bb0ad1bea81f60cbbb2c66c2c0ab24ee12db0026.tar.bz2 | |
Fix some occurrences using string by value rather than by const&
Fix issues diagnosed by clang-tidy
- performance-unnecessary-value-param
- performance-unnecessary-copy-initialization
Signed-off-by: Matthias Maennich <matthias@maennich.net>
Diffstat (limited to 'Source/cmGlobalXCodeGenerator.cxx')
| -rw-r--r-- | Source/cmGlobalXCodeGenerator.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Source/cmGlobalXCodeGenerator.cxx b/Source/cmGlobalXCodeGenerator.cxx index e9950d9..a1bec6b 100644 --- a/Source/cmGlobalXCodeGenerator.cxx +++ b/Source/cmGlobalXCodeGenerator.cxx @@ -573,7 +573,7 @@ void cmGlobalXCodeGenerator::ClearXCodeObjects() void cmGlobalXCodeGenerator::addObject(cmXCodeObject* obj) { if (obj->GetType() == cmXCodeObject::OBJECT) { - std::string id = obj->GetId(); + const std::string& id = obj->GetId(); // If this is a duplicate id, it's an error: // @@ -2748,7 +2748,7 @@ bool cmGlobalXCodeGenerator::CreateGroups( } cmXCodeObject* cmGlobalXCodeGenerator::CreatePBXGroup(cmXCodeObject* parent, - std::string name) + const std::string& name) { cmXCodeObject* parentChildren = nullptr; if (parent) |