diff options
| author | Christian Stenger <christian.stenger@theqtcompany.com> | 2015-09-04 10:57:05 +0200 |
|---|---|---|
| committer | Christian Stenger <christian.stenger@theqtcompany.com> | 2015-10-02 10:42:21 +0300 |
| commit | 0d01a0c7bfe2c80700b64e1b56c1172cc99a6f45 (patch) | |
| tree | 885780978dcfa4fe1efc6df9a2ea243477006c3c | |
| parent | 274d095f46fd85aecc56645bc7ce44ba0dd5a653 (diff) | |
| download | qt-creator-0d01a0c7bfe2c80700b64e1b56c1172cc99a6f45.tar.gz | |
Simplify check state handling
Data functions and special functions cannot be checked at all, so
there is no need to inherit the state by its parent.
Change-Id: If72f79ef6bcd77628916bd7fbe137faa5555db41
Reviewed-by: Niels Weber <niels.weber@theqtcompany.com>
Reviewed-by: David Schulz <david.schulz@theqtcompany.com>
| -rw-r--r-- | plugins/autotest/testtreeitem.cpp | 13 |
1 files changed, 1 insertions, 12 deletions
diff --git a/plugins/autotest/testtreeitem.cpp b/plugins/autotest/testtreeitem.cpp index 721c605473..5119f4a206 100644 --- a/plugins/autotest/testtreeitem.cpp +++ b/plugins/autotest/testtreeitem.cpp @@ -30,21 +30,10 @@ TestTreeItem::TestTreeItem(const QString &name, const QString &filePath, Type ty m_parent(parent) { switch (m_type) { - case ROOT: - m_checked = Qt::Unchecked; - break; case TEST_CLASS: case TEST_FUNCTION: m_checked = Qt::Checked; break; - case TEST_DATAFUNCTION: - case TEST_SPECIALFUNCTION: - if (m_parent) - m_checked = m_parent->checked() == Qt::PartiallyChecked ? Qt::Unchecked - : m_parent->checked(); - else - m_checked = Qt::Unchecked; - break; default: m_checked = Qt::Unchecked; } @@ -169,7 +158,7 @@ Qt::CheckState TestTreeItem::checked() const return m_checked; case TEST_DATAFUNCTION: case TEST_SPECIALFUNCTION: - return m_parent->m_checked == Qt::PartiallyChecked ? Qt::Unchecked : m_parent->m_checked; + return Qt::Unchecked; default: if (m_parent) return m_parent->m_checked; |