diff options
| author | Brad King <brad.king@kitware.com> | 2016-05-27 09:08:21 -0400 |
|---|---|---|
| committer | CMake Topic Stage <kwrobot@kitware.com> | 2016-05-27 09:08:21 -0400 |
| commit | fb1f5d50af2bc97b8dfe783998a66a7a4bea140e (patch) | |
| tree | 1b55b11e2626dbf1dc948cce00094d528dbde7c1 /Source/CTest/cmParseBlanketJSCoverage.cxx | |
| parent | 82c286393afb15346431a8a3f601819698fcb87f (diff) | |
| parent | 27ead963052b4c3f4e40ea9e6141ba9902ad310a (diff) | |
| download | cmake-fb1f5d50af2bc97b8dfe783998a66a7a4bea140e.tar.gz | |
Merge topic 'remove-needless-copies'
27ead963 Remove unnecessary local copies.
618fb23f Pass arguments that are not modified as const&.
Diffstat (limited to 'Source/CTest/cmParseBlanketJSCoverage.cxx')
| -rw-r--r-- | Source/CTest/cmParseBlanketJSCoverage.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Source/CTest/cmParseBlanketJSCoverage.cxx b/Source/CTest/cmParseBlanketJSCoverage.cxx index 152c354aca..fa539e45b5 100644 --- a/Source/CTest/cmParseBlanketJSCoverage.cxx +++ b/Source/CTest/cmParseBlanketJSCoverage.cxx @@ -46,7 +46,7 @@ public: return line.substr(begIndex, line.npos); } } - bool ParseFile(std::string file) + bool ParseFile(std::string const& file) { FileLinesType localCoverageVector; std::string filename; @@ -138,7 +138,7 @@ bool cmParseBlanketJSCoverage::LoadCoverageData(std::vector<std::string> files) return true; } -bool cmParseBlanketJSCoverage::ReadJSONFile(std::string file) +bool cmParseBlanketJSCoverage::ReadJSONFile(std::string const& file) { cmParseBlanketJSCoverage::JSONParser parser(this->Coverage); cmCTestOptionalLog(this->CTest, HANDLER_VERBOSE_OUTPUT, |