remove redundant call to updateBreakpointStatus() on removing breakpoints, fixing the bug that on second debug session breakpoints are not marked as resolved.
authorl12wang
Thu, 15 Oct 2009 15:38:06 -0500
changeset 104 b23e845b9366
parent 103 1b990d2a43dd
child 105 fbceb3d6fb44
remove redundant call to updateBreakpointStatus() on removing breakpoints, fixing the bug that on second debug session breakpoints are not marked as resolved.
cdt/cdt_6_0_x/org.eclipse.cdt.dsf/src/org/eclipse/cdt/dsf/debug/service/BreakpointsMediator.java
--- a/cdt/cdt_6_0_x/org.eclipse.cdt.dsf/src/org/eclipse/cdt/dsf/debug/service/BreakpointsMediator.java	Wed Oct 14 15:28:00 2009 -0500
+++ b/cdt/cdt_6_0_x/org.eclipse.cdt.dsf/src/org/eclipse/cdt/dsf/debug/service/BreakpointsMediator.java	Thu Oct 15 15:38:06 2009 -0500
@@ -341,25 +341,7 @@
         countingRm.setDoneCount(platformBPs.size());
 
         for (final IBreakpoint bp : platformBPs.keySet()) {
-            uninstallBreakpoint(dmc, bp, 
-                new RequestMonitor(getExecutor(), countingRm) {
-                    @Override
-                    protected void handleCompleted() {
-                        // After the breakpoint is removed from target.  Call the attribute 
-                        // translator to refresh breakpoint status based on the new target 
-                        // breakpoint status. 
-                        new Job("Breakpoint status update") { //$NON-NLS-1$
-                            { setSystem(true); }
-                            @Override
-                            protected IStatus run(IProgressMonitor monitor) {
-                                fAttributeTranslator.updateBreakpointStatus(bp, EBreakpointStatusChange.EUninstalled);
-                                return Status.OK_STATUS;
-                            };
-                        }.schedule();
-
-                        countingRm.done();
-                    }
-                });
+            uninstallBreakpoint(dmc, bp, countingRm); 
         }
     }