Skip to content

Commit b51e34b

Browse files
vogellaclaude
andcommitted
Fix ECJ type inference error in lambda test fixtures
Add explicit type witness to Arrays.sort calls to avoid ECJ type inference ambiguity with Comparator<? super T> wildcards. The PR's change to use BUILD_RELEVANT_NAMESPACES in addTransitiveDependenciesWithForbiddenAccess changes the classpath composition, exposing a stricter ECJ type inference behavior for lambdas with explicitly typed parameters. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 3123b7c commit b51e34b

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

apitools/org.eclipse.pde.api.tools.tests/test-builder/usage/java8/lambda/testLambdaExpression.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@ public class testLambdaExpression {
2727
void m1() {
2828
FieldUsageClass noRef = new FieldUsageClass();
2929
String[] array = {"one"};
30-
Arrays.sort(array, (String s1, String s2) -> noRef.f1);
31-
Arrays.sort(array, (String s1, String s2) -> FieldUsageClass.f2);
30+
Arrays.<String>sort(array, (String s1, String s2) -> noRef.f1);
31+
Arrays.<String>sort(array, (String s1, String s2) -> FieldUsageClass.f2);
3232
}
3333

3434
}

apitools/org.eclipse.pde.api.tools.tests/test-builder/usage/java8/lambda/testLambdaStatement.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@ public class testLambdaStatement {
2727
void m1() {
2828
FieldUsageClass noRef = new FieldUsageClass();
2929
String[] array = {"one"};
30-
Arrays.sort(array, (String s1, String s2) -> noRef.f1);
31-
Arrays.sort(array, (String s1, String s2) -> FieldUsageClass.f2);
30+
Arrays.<String>sort(array, (String s1, String s2) -> noRef.f1);
31+
Arrays.<String>sort(array, (String s1, String s2) -> FieldUsageClass.f2);
3232
}
3333

3434
}

0 commit comments

Comments
 (0)