From 0b291f4f12b61afa503a82d9dbfc9f438db7c824 Mon Sep 17 00:00:00 2001
From: Pawel Piech <pawel.piech@windriver.com>
Date: Fri, 22 Feb 2008 00:29:18 +0000
Subject: [PATCH] [219841] Added TODO comments regarding this bug.

---
 .../dd/dsf/debug/service/BreakpointsMediator.java  | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

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<IBreakpointDMContext>();
 						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<IBreakpointDMContext>(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();
                                     }
                                 });