From 9f6cfe480394dd94b4919cf9254bd738600869d2 Mon Sep 17 00:00:00 2001
From: hjk <hjk@theqtcompany.com>
Date: Fri, 3 Jul 2015 14:21:30 +0200
Subject: [PATCH] TreeModel: Assert when inserting items that already have a
 model

Makes the assumption Item::m_model != 0 if and only if item belongs
to a model explicit.

Change-Id: Ib58889fd10e9fce1590a42ade6d847aee747ac56
Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>
---
 src/libs/utils/treemodel.cpp | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/libs/utils/treemodel.cpp b/src/libs/utils/treemodel.cpp
index d761ba5e9cf..149d1a4c001 100644
--- a/src/libs/utils/treemodel.cpp
+++ b/src/libs/utils/treemodel.cpp
@@ -682,6 +682,7 @@ void TreeItem::appendChild(TreeItem *item)
 
 void TreeItem::insertChild(int pos, TreeItem *item)
 {
+    QTC_CHECK(!item->model());
     QTC_CHECK(!item->parent());
     QTC_ASSERT(0 <= pos && pos <= m_children.size(), return); // '<= size' is intentional.