summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorOrgad Shaneh <orgad.shaneh@audiocodes.com>2014-08-24 08:36:07 +0300
committerOrgad Shaneh <orgads@gmail.com>2014-08-25 11:01:06 +0200
commit5b5a7d1deaa77aac0e3c3c3e89523ee9562bc4a6 (patch)
treec651ffd0bafc0a7c97b99b2a1b45d632b6fc7386
parent72c60097c3957be617f2e1c9dc5897862b931eff (diff)
downloadqt-creator-5b5a7d1deaa77aac0e3c3c3e89523ee9562bc4a6.tar.gz
Debugger: Avoid duplication of SDK debuggers
Task-number: QTCREATORBUG-12888 Change-Id: I732f1bfe44dc47d0c96570649f3bf8c0e3f40dda Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>
-rw-r--r--src/plugins/debugger/debuggeritemmanager.cpp9
1 files changed, 3 insertions, 6 deletions
diff --git a/src/plugins/debugger/debuggeritemmanager.cpp b/src/plugins/debugger/debuggeritemmanager.cpp
index 05a6b428ea..52c1453e37 100644
--- a/src/plugins/debugger/debuggeritemmanager.cpp
+++ b/src/plugins/debugger/debuggeritemmanager.cpp
@@ -378,18 +378,15 @@ void DebuggerItemManager::saveDebuggers()
QVariant DebuggerItemManager::registerDebugger(const DebuggerItem &item)
{
- // Force addition when Id is set.
- if (item.id().isValid())
- return addDebugger(item);
-
- // Otherwise, try re-using existing item first.
+ // Try re-using existing item first.
foreach (const DebuggerItem &d, m_debuggers) {
if (d.command() == item.command()
&& d.isAutoDetected() == item.isAutoDetected()
&& d.engineType() == item.engineType()
&& d.displayName() == item.displayName()
- && d.abis() == item.abis())
+ && d.abis() == item.abis()) {
return d.id();
+ }
}
// Nothing suitable. Create a new id and add the item.