Attachment 205458 Details for Bug 345298 – Patch with suggested fix.

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] Patch with suggested fix.

Bug-345298--breakpoints-Attempting-to-launch-JS-debu.patch (text/plain), 1.89 KB, created by Pawel Piech

on 2011-10-18 16:27:29 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Pawel Piech

Created: 2011-10-18 16:27:29 EDT

Size: 1.89 KB

patch

obsolete

>diff --git a/org.eclipse.debug.core/core/org/eclipse/debug/core/model/Breakpoint.java b/org.eclipse.debug.core/core/org/eclipse/debug/core/model/Breakpoint.java
>index 3fd04ff..85461f7 100644
>--- a/org.eclipse.debug.core/core/org/eclipse/debug/core/model/Breakpoint.java
>+++ b/org.eclipse.debug.core/core/org/eclipse/debug/core/model/Breakpoint.java
>@@ -41,17 +41,21 @@
> 
> public abstract class Breakpoint extends PlatformObject implements IBreakpoint {
> 	
>-	static {
>-		// making sure that the BreakpointManager is correctly initialized
>-		// before any breakpoint marker related operation (see bug 54993)
>-		DebugPlugin.getDefault().getBreakpointManager().getBreakpoints();
>-	}
>-				
> 	/**
> 	 * Underlying marker.
> 	 */
> 	private IMarker fMarker= null;
>-	
>+
>+    /**
>+     * Default constructor used to ensure that breakpoint manager is initialized.
>+     */
>+    public Breakpoint() {
>+        // making sure that the BreakpointManager is correctly initialized
>+        // before any breakpoint marker related operation (see bug 54993)
>+        DebugPlugin.getDefault().getBreakpointManager().getBreakpoints();
>+    }
>+                
>+
> 	/**
> 	 * @see IBreakpoint#setMarker(IMarker)
> 	 */
>diff --git a/org.eclipse.debug.core/core/org/eclipse/debug/internal/core/BreakpointManager.java b/org.eclipse.debug.core/core/org/eclipse/debug/internal/core/BreakpointManager.java
>index 4d631c2..cb666a0 100644
>--- a/org.eclipse.debug.core/core/org/eclipse/debug/internal/core/BreakpointManager.java
>+++ b/org.eclipse.debug.core/core/org/eclipse/debug/internal/core/BreakpointManager.java
>@@ -394,6 +394,13 @@
> 	}
> 	
> 	/**
>+	 * Calls {@link #initializeBreakpoints()} if breakpoints have not yet been initialized.
>+	 */
>+	public void ensureBreakpointsInitialized() {
>+	    getBreakpoints0();
>+	}
>+	
>+	/**
> 	 * The BreakpointManager waits to load the breakpoints
> 	 * of the workspace until a request is made to retrieve the 
> 	 * breakpoints.

Actions: View | Diff

Attachments on bug 345298: 195241 | 195244 | 195245 | 195260 | 195261 | 205458 | 206104 | 207048 | 207052