remove redundant call to updateBreakpointStatus() on removing breakpoints, fixing the bug that on second debug session breakpoints are not marked as resolved.
--- 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);
}
}