From 90ad8fb3521c41f246d8e24eaeb9dcb3d57cfdc8 Mon Sep 17 00:00:00 2001 From: Ken Ryall Date: Thu, 17 Aug 2006 23:44:10 +0000 Subject: [PATCH] If the breakpoint manager is disabled disable the breakpoint using the same method (async exec) as doHandleLocationBreakpointCreatedEvent. Otherwise it's async thread may enable the breakpoint again after this code runs. --- .../cdt/debug/internal/core/CBreakpointManager.java | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/debug/org.eclipse.cdt.debug.core/src/org/eclipse/cdt/debug/internal/core/CBreakpointManager.java b/debug/org.eclipse.cdt.debug.core/src/org/eclipse/cdt/debug/internal/core/CBreakpointManager.java index 173b81472c6..5f7083bcdac 100644 --- a/debug/org.eclipse.cdt.debug.core/src/org/eclipse/cdt/debug/internal/core/CBreakpointManager.java +++ b/debug/org.eclipse.cdt.debug.core/src/org/eclipse/cdt/debug/internal/core/CBreakpointManager.java @@ -407,12 +407,7 @@ public class CBreakpointManager implements IBreakpointsListener, IBreakpointMana else if ( cdiBreakpoint instanceof ICDILocationBreakpoint ) doHandleLocationBreakpointCreatedEvent( (ICDILocationBreakpoint)cdiBreakpoint ); if ( !cdiBreakpoint.isTemporary() && !DebugPlugin.getDefault().getBreakpointManager().isEnabled() ) { - try { - cdiBreakpoint.setEnabled( false ); - } - catch( CDIException e ) { - // ignore - } + changeBreakpointPropertiesOnTarget(cdiBreakpoint, new Boolean(false), null); } }