diff options
author | Daniel Pfeifer <daniel@pfeifer-mail.de> | 2016-08-18 18:04:21 (GMT) |
---|---|---|
committer | Daniel Pfeifer <daniel@pfeifer-mail.de> | 2016-08-18 18:04:21 (GMT) |
commit | 50ad1e0a144ae1f2267a4966789e5a16372f458e (patch) | |
tree | ab43b965115505a6d7367566d280b2d9b179ff11 /Source/CTest/cmCTestSVN.cxx | |
parent | 7f97a6c94b59be7e7eba7362ce3eecdcff79ab70 (diff) | |
download | CMake-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.cxx | 9 |
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, |