From 064ce187b6de00e343b26a614a98846ef46213d7 Mon Sep 17 00:00:00 2001 From: Marc-Andre Laperle Date: Tue, 28 Feb 2017 09:04:46 -0500 Subject: [PATCH] lldb: Add some comments Change-Id: I82a825e8a3d95e895131857d3217c71be2072208 Signed-off-by: Marc-Andre Laperle --- .../cdt/llvm/dsf/lldb/core/internal/service/LLDBProcesses.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/llvm/org.eclipse.cdt.llvm.dsf.lldb.core/src/org/eclipse/cdt/llvm/dsf/lldb/core/internal/service/LLDBProcesses.java b/llvm/org.eclipse.cdt.llvm.dsf.lldb.core/src/org/eclipse/cdt/llvm/dsf/lldb/core/internal/service/LLDBProcesses.java index 97ccb1869d4..06da8c1788a 100644 --- a/llvm/org.eclipse.cdt.llvm.dsf.lldb.core/src/org/eclipse/cdt/llvm/dsf/lldb/core/internal/service/LLDBProcesses.java +++ b/llvm/org.eclipse.cdt.llvm.dsf.lldb.core/src/org/eclipse/cdt/llvm/dsf/lldb/core/internal/service/LLDBProcesses.java @@ -59,7 +59,7 @@ public class LLDBProcesses extends GDBProcesses_7_4 { @Override public void getRunningProcesses(IDMContext dmc, DataRequestMonitor rm) { - // '-list-thread-groups --available' is not supported by lldm-mi so we + // FIXME '-list-thread-groups --available' is not supported by lldm-mi so we // fall back to CDT Core's implementation of listing running processes. // This works just like GDBProcesses#getRunningProcesses. @@ -115,6 +115,7 @@ public class LLDBProcesses extends GDBProcesses_7_4 { // yet so the process names map will be empty. This can happen when // doing local debugging but not attach mode. if (fProcessNames.isEmpty()) { + // FIXME: This triggers a double done()!! (Bug 510833) getRunningProcesses(dmc, new DataRequestMonitor<>(getExecutor(), rm)); }