Bugfix for #518 (Setting a breakpoint inside lambda with object).

eclipse-jdt · Dec 12, 2024 · 7cebcbd · 7cebcbd
1 parent a9fe90d
commit 7cebcbd
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 9 deletions.
diff --git a/org.eclipse.jdt.debug.tests/java8/ClassWithLambdas.java b/org.eclipse.jdt.debug.tests/java8/ClassWithLambdas.java
@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2024 IBM Corporation and others.
+ * Copyright (c) 2024 Christian Schima.
  *
  * This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License 2.0

diff --git a/...model/org/eclipse/jdt/internal/debug/core/breakpoints/ValidBreakpointLocationLocator.java b/...model/org/eclipse/jdt/internal/debug/core/breakpoints/ValidBreakpointLocationLocator.java
@@ -1067,14 +1067,8 @@ public boolean visit(LambdaExpression node) {
 						}
 					}
 
-				// Lambda body can be a block (which is handled above) or an (arbitrary) expression. So maybe the
-				// following cases are insufficient and should be replaced by handling the general
-				// org.eclipse.jdt.core.dom.Expression.
-				} else if (body instanceof LambdaExpression) {
-					body.accept(this);
-				} else if (body instanceof MethodInvocation) {
-					body.accept(this);
-				} else if (body instanceof ClassInstanceCreation) {
+					// Lambda body can be a block (which is handled above) or an (arbitrary) expression.
+				} else if (body instanceof Expression) {
 					body.accept(this);
 				}
 			}