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/projectexplorer/toolchainoptionspage.cpp | |
| parent | 5309e217e460d4f20ab932334b9e808216f9b2a7 (diff) | |
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/projectexplorer/toolchainoptionspage.cpp')
| -rw-r--r-- | src/plugins/projectexplorer/toolchainoptionspage.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/plugins/projectexplorer/toolchainoptionspage.cpp b/src/plugins/projectexplorer/toolchainoptionspage.cpp index 6482acebdb..54fabc7866 100644 --- a/src/plugins/projectexplorer/toolchainoptionspage.cpp +++ b/src/plugins/projectexplorer/toolchainoptionspage.cpp @@ -259,8 +259,7 @@ void ToolChainOptionsWidget::removeToolChain(ToolChain *tc) TreeItem *parent = m_model.rootItem()->child(tc->isAutoDetected() ? 0 : 1); foreach (ToolChainTreeItem *item, m_model.itemsAtLevel<ToolChainTreeItem *>(1, parent)) { if (item->toolChain == tc) { - m_model.takeItem(item); - delete item; + delete m_model.takeItem(item); break; } } |