diff options
| author | Eike Ziller <eike.ziller@theqtcompany.com> | 2015-06-09 09:33:14 +0200 |
|---|---|---|
| committer | Eike Ziller <eike.ziller@theqtcompany.com> | 2015-06-10 08:05:32 +0000 |
| commit | b1393d4778056f218f8a02cb529919a08e211fd5 (patch) | |
| tree | b5188bbc2b9da64f2e0a9587333fc00ca3992a00 /src/plugins/debugger/debuggeroptionspage.cpp | |
| parent | 5309e217e460d4f20ab932334b9e808216f9b2a7 (diff) | |
| download | qt-creator-b1393d4778056f218f8a02cb529919a08e211fd5.tar.gz | |
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>
Diffstat (limited to 'src/plugins/debugger/debuggeroptionspage.cpp')
| -rw-r--r-- | src/plugins/debugger/debuggeroptionspage.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
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); } |