From b1393d4778056f218f8a02cb529919a08e211fd5 Mon Sep 17 00:00:00 2001
From: Eike Ziller <eike.ziller@theqtcompany.com>
Date: Tue, 9 Jun 2015 09:33:14 +0200
Subject: TreeModel: takeItem should return the item

The usual Qt API for "takeXyz()" methods is to return the item that was
"taken".

Change-Id: Ie144051801487a301b3f13e2857735b65f58150b
Reviewed-by: hjk <hjk@theqtcompany.com>
---
 src/plugins/debugger/debuggeroptionspage.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

(limited to 'src/plugins/debugger/debuggeroptionspage.cpp')

diff --git a/src/plugins/debugger/debuggeroptionspage.cpp b/src/plugins/debugger/debuggeroptionspage.cpp
index 00198db40e..1715cc9cbe 100644
--- a/src/plugins/debugger/debuggeroptionspage.cpp
+++ b/src/plugins/debugger/debuggeroptionspage.cpp
@@ -165,8 +165,7 @@ void DebuggerItemModel::removeCurrentDebugger()
     QVariant id = m_currentTreeItem->m_item.id();
     DebuggerTreeItem *treeItem = m_currentTreeItem;
     m_currentTreeItem = 0;
-    takeItem(treeItem);
-    delete treeItem;
+    delete takeItem(treeItem);
     m_removedItems.append(id);
 }
 
-- 
cgit v1.2.1