Optimize to avoid redundant getMarker() calls

Copilot · carstenartur · Copilot · commit 132c87827567 · 2026-02-01T08:14:03.000Z
Co-authored-by: carstenartur &lt;3164220+carstenartur@users.noreply.github.com&gt;
diff --git a/org.eclipse.jdt.debug/model/org/eclipse/jdt/internal/debug/core/breakpoints/JavaBreakpoint.java b/org.eclipse.jdt.debug/model/org/eclipse/jdt/internal/debug/core/breakpoints/JavaBreakpoint.java
@@ -286,7 +286,8 @@ protected void deregisterRequest(EventRequest request, JDIDebugTarget target)
 		// deletion.
 		// Don't try updating the marker (decrementing the install count) if
 		// it no longer exists.
-		if (!(request instanceof ClassPrepareRequest) && getMarker() != null && getMarker().exists()) {
+		IMarker marker = getMarker();
+		if (!(request instanceof ClassPrepareRequest) && marker != null && marker.exists()) {
 			decrementInstallCount();
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -286,7 +286,8 @@ protected void deregisterRequest(EventRequest request, JDIDebugTarget target)`
`286`	`286`	`// deletion.`
`287`	`287`	`// Don't try updating the marker (decrementing the install count) if`
`288`	`288`	`// it no longer exists.`
`289`		`- if (!(request instanceof ClassPrepareRequest) && getMarker() != null && getMarker().exists()) {`
	`289`	`+ IMarker marker = getMarker();`
	`290`	`+ if (!(request instanceof ClassPrepareRequest) && marker != null && marker.exists()) {`
`290`	`291`	`decrementInstallCount();`
`291`	`292`	`}`
`292`	`293`	`}`