diff options
| author | Daniel Pfeifer <daniel@pfeifer-mail.de> | 2016-10-18 21:16:02 (GMT) |
|---|---|---|
| committer | Brad King <brad.king@kitware.com> | 2016-10-20 12:48:20 (GMT) |
| commit | effa6c8343fb871574cb5471de5eb4c1705dd334 (patch) | |
| tree | 9349fd3223bf5d9369e80eace97ad04209c52bd1 /Source/cmTarget.cxx | |
| parent | fb461cacba5afe98fd9871d4b8ab0500d825ad5a (diff) | |
| download | CMake-effa6c8343fb871574cb5471de5eb4c1705dd334.zip CMake-effa6c8343fb871574cb5471de5eb4c1705dd334.tar.gz CMake-effa6c8343fb871574cb5471de5eb4c1705dd334.tar.bz2 | |
fix more issues reported by clang-tidy
Diffstat (limited to 'Source/cmTarget.cxx')
| -rw-r--r-- | Source/cmTarget.cxx | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/Source/cmTarget.cxx b/Source/cmTarget.cxx index 3a22309..de48193 100644 --- a/Source/cmTarget.cxx +++ b/Source/cmTarget.cxx @@ -852,13 +852,15 @@ void cmTarget::SetProperty(const std::string& prop, const char* value) e << "NAME property is read-only\n"; this->Makefile->IssueMessage(cmake::FATAL_ERROR, e.str()); return; - } else if (prop == "EXPORT_NAME" && this->IsImported()) { + } + if (prop == "EXPORT_NAME" && this->IsImported()) { std::ostringstream e; e << "EXPORT_NAME property can't be set on imported targets (\"" << this->Name << "\")\n"; this->Makefile->IssueMessage(cmake::FATAL_ERROR, e.str()); return; - } else if (prop == "SOURCES" && this->IsImported()) { + } + if (prop == "SOURCES" && this->IsImported()) { std::ostringstream e; e << "SOURCES property can't be set on imported targets (\"" << this->Name << "\")\n"; @@ -932,13 +934,15 @@ void cmTarget::AppendProperty(const std::string& prop, const char* value, e << "NAME property is read-only\n"; this->Makefile->IssueMessage(cmake::FATAL_ERROR, e.str()); return; - } else if (prop == "EXPORT_NAME" && this->IsImported()) { + } + if (prop == "EXPORT_NAME" && this->IsImported()) { std::ostringstream e; e << "EXPORT_NAME property can't be set on imported targets (\"" << this->Name << "\")\n"; this->Makefile->IssueMessage(cmake::FATAL_ERROR, e.str()); return; - } else if (prop == "SOURCES" && this->IsImported()) { + } + if (prop == "SOURCES" && this->IsImported()) { std::ostringstream e; e << "SOURCES property can't be set on imported targets (\"" << this->Name << "\")\n"; |