diff options
author | Daniel Pfeifer <daniel@pfeifer-mail.de> | 2016-05-26 21:58:51 +0200 |
---|---|---|
committer | Daniel Pfeifer <daniel@pfeifer-mail.de> | 2016-05-26 21:58:51 +0200 |
commit | 618fb23fc9838d344e2033c64bfc1a3a55bb7f61 (patch) | |
tree | abaabfaae8a9f5448d7ee062c757c61254792995 /Source/cmSourceGroup.cxx | |
parent | acd8a73044e879286f0cfa24b54497a8307f204b (diff) | |
download | cmake-618fb23fc9838d344e2033c64bfc1a3a55bb7f61.tar.gz |
Pass arguments that are not modified as const&.
Use clang-tidy's performance-unnecessary-value-param checker to find
value parameter declarations of expensive to copy types that are not
modified inside the function. Ignore findings in kwsys.
After applying the fix-its, manually change `const T&` to `T const&`.
Diffstat (limited to 'Source/cmSourceGroup.cxx')
-rw-r--r-- | Source/cmSourceGroup.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Source/cmSourceGroup.cxx b/Source/cmSourceGroup.cxx index c09d596c86..394b33a473 100644 --- a/Source/cmSourceGroup.cxx +++ b/Source/cmSourceGroup.cxx @@ -103,7 +103,7 @@ const std::vector<const cmSourceFile*>& cmSourceGroup::GetSourceFiles() const return this->SourceFiles; } -void cmSourceGroup::AddChild(cmSourceGroup child) +void cmSourceGroup::AddChild(cmSourceGroup const& child) { this->Internal->GroupChildren.push_back(child); } |