Method Breakpoint does not recognised when set at end of a method

This PR fixes the issue where breakpoints set at the end of a method did
not behave like method breakpoints, ensuring both entry and exit events
are consistently shown.

Fix: #697

Author: subyssurendran666

org.eclipse.jdt.debug.ui/ui/org/eclipse/jdt/internal/debug/ui/actions/ToggleBreakpointAdapter.java

@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2000, 2022 IBM Corporation and others.
+ * Copyright (c) 2000, 2025 IBM Corporation and others.
  *
  * This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License 2.0
@@ -402,6 +402,9 @@ private static void doToggleMethodBreakpoint(IMethod member, String lambdaMethod
 				deleteTracepoint(breakpoint, part, monitor);
 				BreakpointToggleUtils.setUnsetTracepoints(false);
 			} else {
+				if (ValidBreakpointLocationLocator.LOCATION_METHOD_CLOSE) {
+					ValidBreakpointLocationLocator.LOCATION_METHOD_CLOSE = false;
+				}
 				deleteBreakpoint(breakpoint, part, monitor);
 			}
 			return;
@@ -445,6 +448,11 @@ private static void doToggleMethodBreakpoint(IMethod member, String lambdaMethod
 			}
 			BreakpointToggleUtils.setUnsetTracepoints(false);
 		}
+		if (ValidBreakpointLocationLocator.LOCATION_METHOD_CLOSE) {
+			methodBreakpoint.setEntry(false);
+			methodBreakpoint.setExit(true);
+			ValidBreakpointLocationLocator.LOCATION_METHOD_CLOSE = false;
+		}
 	}
 
 	/**
@@ -1806,4 +1814,4 @@ private static ICompilationUnit getCompilationUnit(ITextEditor editor) {
 		return JavaCore.createCompilationUnitFrom(file);
 	}
 
-}
+}

org.eclipse.jdt.debug/model/org/eclipse/jdt/internal/debug/core/breakpoints/ValidBreakpointLocationLocator.java

@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2003, 2023 IBM Corporation and others.
+ * Copyright (c) 2003, 2025 IBM Corporation and others.
  *
  * This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License 2.0
@@ -128,6 +128,7 @@ public class ValidBreakpointLocationLocator extends ASTVisitor {
 	public static final int LOCATION_METHOD = 2;
 	public static final int LOCATION_FIELD = 3;
 	public static final int LOCATION_LAMBDA_METHOD = 4;
+	public static boolean LOCATION_METHOD_CLOSE = false;
 
 
 	private final CompilationUnit fCompilationUnit;
@@ -1129,10 +1130,9 @@ public boolean visit(MemberValuePair node) {
 	public boolean visit(MethodDeclaration node) {
 		if (visit(node, false)) {
 			if (fBestMatch) {
-				// check if we are on the line which contains the method name
 				int nameOffset = node.getName().getStartPosition();
 				if (lineNumber(nameOffset) == fLineNumber) {
-					if (node.getParent() instanceof AnonymousClassDeclaration){
+					if (node.getParent() instanceof AnonymousClassDeclaration) {
 						fLocationType = LOCATION_NOT_FOUND;
 						fLocationFound = true;
 						return false;
@@ -1142,16 +1142,29 @@ public boolean visit(MethodDeclaration node) {
 					fLocationFound = true;
 					return false;
 				}
-			}
-			// visit only the body
-			Block body = node.getBody();
-			if (body != null) { // body is null for abstract methods
-				body.accept(this);
+
+				Block body = node.getBody();
+				if (body != null) {
+					int bodyStart = body.getStartPosition();
+					int bodyLength = body.getLength();
+					int bodyEnd = bodyStart + bodyLength;
+					int closingBraceOffset = bodyEnd - 1;
+
+					if (lineNumber(closingBraceOffset) == fLineNumber) {
+						fMemberOffset = closingBraceOffset;
+						LOCATION_METHOD_CLOSE = true;
+						fLocationType = LOCATION_METHOD;
+						fLocationFound = true;
+						return false;
+					}
+
+					body.accept(this);
+				}
 			}
 		}
 		return false;
 	}
-
+	
 	/**
 	 * @see org.eclipse.jdt.core.dom.ASTVisitor#visit(org.eclipse.jdt.core.dom.MethodInvocation)
 	 */
@@ -1610,4 +1623,4 @@ public boolean visit(WhileStatement node) {
 		return visit(node, false);
 	}
 
-}
+}