diff --git a/plugins/org.eclipse.dd.dsf.debug/src/org/eclipse/dd/dsf/debug/service/BreakpointsMediator.java b/plugins/org.eclipse.dd.dsf.debug/src/org/eclipse/dd/dsf/debug/service/BreakpointsMediator.java index 74ab060459c..ee8b08bc760 100644 --- a/plugins/org.eclipse.dd.dsf.debug/src/org/eclipse/dd/dsf/debug/service/BreakpointsMediator.java +++ b/plugins/org.eclipse.dd.dsf.debug/src/org/eclipse/dd/dsf/debug/service/BreakpointsMediator.java @@ -427,6 +427,9 @@ public class BreakpointsMediator extends AbstractDsfService implements IBreakpoi list = new LinkedList(); list.add(getData()); breakpointIDs.put(breakpoint, list); + } else { + // TODO (bug 219841): need to add breakpoint error status tracking + // in addition to fBreakpointDMContexts. } installRM.done(); } @@ -628,13 +631,18 @@ public class BreakpointsMediator extends AbstractDsfService implements IBreakpoi oldBpContexts.get(i), new RequestMonitor(getExecutor(), countingRM) { @Override - protected void handleOK() { + protected void handleCompleted() { fBreakpoints.insertBreakpoint( context, attrs, new DataRequestMonitor(getExecutor(), countingRM) { @Override - protected void handleOK() { - newBpContexts.add(getData()); + protected void handleCompleted() { + if (getStatus().isOK()) { + newBpContexts.add(getData()); + } else { + // TODO (bug 219841): need to add breakpoint error status tracking + // in addition to fBreakpointDMContexts. + } countingRM.done(); } });