feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

ProjectExplorer: Simplify code a little bit

Browse Source 
Use an QTC_ASSERT in favor of an if. This may not happen.

Change-Id: I4e8d58210f9caff9b8ae7d0fccf764d8ceeace9e
Reviewed-by: hjk <hjk@theqtcompany.com>
This commit is contained in:
Tobias Hunger
2016-02-02 12:43:57 +01:00
parent 26c7afdfb7
commit 412dd23e0f
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/plugins/projectexplorer/project.cpp
View File

@@ -201,8 +201,7 @@ void Project::addTarget(Target *t)
bool Project::removeTarget(Target *target) bool Project::removeTarget(Target *target)
{ {
if (!target || !d->m_targets.contains(target)) QTC_ASSERT(target && d->m_targets.contains(target), return false);
return false;
if (BuildManager::isBuilding(target)) if (BuildManager::isBuilding(target))
return false; return false;