From 0d01a0c7bfe2c80700b64e1b56c1172cc99a6f45 Mon Sep 17 00:00:00 2001 From: Christian Stenger <christian.stenger@theqtcompany.com> Date: Fri, 4 Sep 2015 10:57:05 +0200 Subject: [PATCH] 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> --- plugins/autotest/testtreeitem.cpp | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/plugins/autotest/testtreeitem.cpp b/plugins/autotest/testtreeitem.cpp index 721c605473e..5119f4a206d 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; -- GitLab