diff options
| author | Matthias Maennich <matthias@maennich.net> | 2017-09-19 14:19:31 (GMT) |
|---|---|---|
| committer | Brad King <brad.king@kitware.com> | 2017-09-28 11:23:42 (GMT) |
| commit | 870dd06da15751250786d435ee4c1361a2b10b4d (patch) | |
| tree | 3a59f41d841e62356b088c9c82f37153f36c003d /Source/cmFileCommand.cxx | |
| parent | 2033abff0dec29072ac070cb6fe1604e8c03f180 (diff) | |
| download | CMake-870dd06da15751250786d435ee4c1361a2b10b4d.zip CMake-870dd06da15751250786d435ee4c1361a2b10b4d.tar.gz CMake-870dd06da15751250786d435ee4c1361a2b10b4d.tar.bz2 | |
Fix left-over occurrences of else-after-return
Fix issues diagnosed by clang-tidy [readability-else-after-return]
These were mostly only showing up on OSX.
Signed-off-by: Matthias Maennich <matthias@maennich.net>
Diffstat (limited to 'Source/cmFileCommand.cxx')
| -rw-r--r-- | Source/cmFileCommand.cxx | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/Source/cmFileCommand.cxx b/Source/cmFileCommand.cxx index f06ef43..fdd5f0c 100644 --- a/Source/cmFileCommand.cxx +++ b/Source/cmFileCommand.cxx @@ -2294,10 +2294,9 @@ bool cmFileCommand::HandleReadElfCommand(std::vector<std::string> const& args) if (errorArg.GetString().empty()) { this->SetError(error); return false; - } else { - this->Makefile->AddDefinition(errorArg.GetString(), error.c_str()); - return true; } + this->Makefile->AddDefinition(errorArg.GetString(), error.c_str()); + return true; #endif } |