diff options
| author | Bill Hoffman <bill.hoffman@kitware.com> | 2015-03-26 15:05:56 (GMT) |
|---|---|---|
| committer | Bill Hoffman <bill.hoffman@kitware.com> | 2015-03-26 15:05:56 (GMT) |
| commit | e680d43d7dd96096962379f627828a182ac0fa80 (patch) | |
| tree | c312ff22bd3dc79c2fd74efaaa98bac7b5cf7cb9 /Source | |
| parent | a4a1b729c63ec81037af06ac1009d12cc1dc3188 (diff) | |
| download | CMake-e680d43d7dd96096962379f627828a182ac0fa80.zip CMake-e680d43d7dd96096962379f627828a182ac0fa80.tar.gz CMake-e680d43d7dd96096962379f627828a182ac0fa80.tar.bz2 | |
Fix an unused variable warning from scanbuild.
Diffstat (limited to 'Source')
| -rw-r--r-- | Source/cmSetTestsPropertiesCommand.cxx | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/Source/cmSetTestsPropertiesCommand.cxx b/Source/cmSetTestsPropertiesCommand.cxx index d079a19..e9cfacc 100644 --- a/Source/cmSetTestsPropertiesCommand.cxx +++ b/Source/cmSetTestsPropertiesCommand.cxx @@ -26,14 +26,12 @@ bool cmSetTestsPropertiesCommand // first collect up the list of files std::vector<std::string> propertyPairs; - bool doingFiles = true; int numFiles = 0; std::vector<std::string>::const_iterator j; for(j= args.begin(); j != args.end();++j) { if(*j == "PROPERTIES") { - doingFiles = false; // now loop through the rest of the arguments, new style ++j; if (std::distance(j, args.end()) % 2 != 0) @@ -44,15 +42,9 @@ bool cmSetTestsPropertiesCommand propertyPairs.insert(propertyPairs.end(), j, args.end()); break; } - else if (doingFiles) - { - numFiles++; - } else { - this->SetError("called with illegal arguments, maybe " - "missing a PROPERTIES specifier?"); - return false; + numFiles++; } } if(propertyPairs.empty()) @@ -62,7 +54,6 @@ bool cmSetTestsPropertiesCommand return false; } - // now loop over all the targets int i; for(i = 0; i < numFiles; ++i) |