summaryrefslogtreecommitdiffstats
path: root/Source/CTest/cmCTestSVN.cxx
diff options
context:
space:
mode:
authorDaniel Pfeifer <daniel@pfeifer-mail.de>2016-08-18 18:04:21 (GMT)
committerDaniel Pfeifer <daniel@pfeifer-mail.de>2016-08-18 18:04:21 (GMT)
commit50ad1e0a144ae1f2267a4966789e5a16372f458e (patch)
treeab43b965115505a6d7367566d280b2d9b179ff11 /Source/CTest/cmCTestSVN.cxx
parent7f97a6c94b59be7e7eba7362ce3eecdcff79ab70 (diff)
downloadCMake-50ad1e0a144ae1f2267a4966789e5a16372f458e.zip
CMake-50ad1e0a144ae1f2267a4966789e5a16372f458e.tar.gz
CMake-50ad1e0a144ae1f2267a4966789e5a16372f458e.tar.bz2
CTest: don't use else after return
Diffstat (limited to 'Source/CTest/cmCTestSVN.cxx')
-rw-r--r--Source/CTest/cmCTestSVN.cxx9
1 files changed, 4 insertions, 5 deletions
diff --git a/Source/CTest/cmCTestSVN.cxx b/Source/CTest/cmCTestSVN.cxx
index 06f838c..bee8296 100644
--- a/Source/CTest/cmCTestSVN.cxx
+++ b/Source/CTest/cmCTestSVN.cxx
@@ -80,11 +80,11 @@ static bool cmCTestSVNPathStarts(std::string const& p1, std::string const& p2)
// Does path p1 start with path p2?
if (p1.size() == p2.size()) {
return p1 == p2;
- } else if (p1.size() > p2.size() && p1[p2.size()] == '/') {
+ }
+ if (p1.size() > p2.size() && p1[p2.size()] == '/') {
return strncmp(p1.c_str(), p2.c_str(), p2.size()) == 0;
- } else {
- return false;
}
+ return false;
}
std::string cmCTestSVN::LoadInfo(SVNInfo& svninfo)
@@ -295,9 +295,8 @@ bool cmCTestSVN::RunSVNCommand(std::vector<char const*> const& parameters,
if (strcmp(parameters[0], "update") == 0) {
return RunUpdateCommand(&args[0], out, err);
- } else {
- return RunChild(&args[0], out, err);
}
+ return RunChild(&args[0], out, err);
}
class cmCTestSVN::LogParser : public cmCTestVC::OutputLogger,