Skip to content

Commit afceb13

Browse files
Rawi01rspilker
Rawi01
authored and
rspilker
committed
[fixes #3373] Find references for extension methods
1 parent d56b576 commit afceb13

13 files changed

Lines changed: 184 additions & 2 deletions

File tree

‎src/eclipseAgent/lombok/eclipse/agent/EclipsePatcher.java‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -918,6 +918,13 @@ private static void patchExtensionMethod(ScriptManager sm) {
918918
.request(StackRequest.THIS)
919919
.wrapMethod(new Hook(PATCH_EXTENSIONMETHOD_COMPLETIONPROPOSAL_PORTAL, "getJavaCompletionProposals", I_JAVA_COMPLETION_PROPOSAL_SIG, "java.lang.Object[]", "java.lang.Object"))
920920
.build());
921+
922+
sm.addScriptIfWitness(OSGI_TYPES, ScriptBuilder.wrapReturnValue()
923+
.target(new MethodTarget("org.eclipse.jdt.core.search.SearchPattern", "createPattern", "org.eclipse.jdt.core.search.SearchPattern", "org.eclipse.jdt.core.IJavaElement", "int", "int"))
924+
.wrapMethod(new Hook(PATCH_EXTENSIONMETHOD, "modifyMethodPattern", "java.lang.Object", "java.lang.Object"))
925+
.cast()
926+
.request(StackRequest.RETURN_VALUE)
927+
.build());
921928
}
922929

923930
private static void patchNullCheck(ScriptManager sm) {

‎src/eclipseAgent/lombok/eclipse/agent/PatchExtensionMethod.java‎

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@
4343
import lombok.experimental.ExtensionMethod;
4444
import lombok.permit.Permit;
4545

46+
import org.eclipse.jdt.core.search.SearchPattern;
4647
import org.eclipse.jdt.internal.compiler.ast.ASTNode;
4748
import org.eclipse.jdt.internal.compiler.ast.Annotation;
4849
import org.eclipse.jdt.internal.compiler.ast.ClassLiteralAccess;
@@ -66,6 +67,7 @@
6667
import org.eclipse.jdt.internal.compiler.lookup.TypeBinding;
6768
import org.eclipse.jdt.internal.compiler.lookup.TypeIds;
6869
import org.eclipse.jdt.internal.compiler.problem.ProblemReporter;
70+
import org.eclipse.jdt.internal.core.search.matching.MethodPattern;
6971

7072
public class PatchExtensionMethod {
7173
static class Extension {
@@ -378,6 +380,17 @@ public static TypeBinding resolveType(TypeBinding resolvedType, MessageSend meth
378380
MessageSend_postponedErrors.clear(methodCall);
379381
return resolvedType;
380382
}
383+
384+
public static SearchPattern modifyMethodPattern(SearchPattern original) {
385+
if (original != null && original instanceof MethodPattern) {
386+
MethodPattern methodPattern = (MethodPattern) original;
387+
if (methodPattern.parameterCount > 0) {
388+
methodPattern.varargs = true;
389+
}
390+
}
391+
392+
return original;
393+
}
381394

382395
private static boolean requiresPolyBinding(Expression argument) {
383396
return Reflection.isFunctionalExpression(argument) || argument instanceof ConditionalExpression && Reflection.isPolyExpression(argument);

‎src/eclipseAgent/lombok/launch/PatchFixesHider.java‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -344,6 +344,7 @@ public static final class ExtensionMethod {
344344
private static final Method ERROR_NO_METHOD_FOR;
345345
private static final Method INVALID_METHOD, INVALID_METHOD2;
346346
private static final Method NON_STATIC_ACCESS_TO_STATIC_METHOD;
347+
private static final Method MODIFY_METHOD_PATTERN;
347348

348349
static {
349350
Class<?> shadowed = Util.shadowLoadClass("lombok.eclipse.agent.PatchExtensionMethod");
@@ -352,6 +353,7 @@ public static final class ExtensionMethod {
352353
INVALID_METHOD = Util.findMethod(shadowed, "invalidMethod", PROBLEM_REPORTER_SIG, MESSAGE_SEND_SIG, METHOD_BINDING_SIG);
353354
INVALID_METHOD2 = Util.findMethod(shadowed, "invalidMethod", PROBLEM_REPORTER_SIG, MESSAGE_SEND_SIG, METHOD_BINDING_SIG, SCOPE_SIG);
354355
NON_STATIC_ACCESS_TO_STATIC_METHOD = Util.findMethod(shadowed, "nonStaticAccessToStaticMethod", PROBLEM_REPORTER_SIG, AST_NODE_SIG, METHOD_BINDING_SIG, MESSAGE_SEND_SIG);
356+
MODIFY_METHOD_PATTERN = Util.findMethod(shadowed, "modifyMethodPattern", "org.eclipse.jdt.core.search.SearchPattern");
355357
}
356358

357359
public static Object resolveType(Object resolvedType, Object methodCall, Object scope) {
@@ -373,6 +375,10 @@ public static void invalidMethod(Object problemReporter, Object messageSend, Obj
373375
public static void nonStaticAccessToStaticMethod(Object problemReporter, Object location, Object method, Object messageSend) {
374376
Util.invokeMethod(NON_STATIC_ACCESS_TO_STATIC_METHOD, problemReporter, location, method, messageSend);
375377
}
378+
379+
public static Object modifyMethodPattern(Object original) {
380+
return Util.invokeMethod(MODIFY_METHOD_PATTERN, original);
381+
}
376382
}
377383

378384
/** Contains patch code to support Javadoc for generated methods */

‎test/eclipse/resource/findreferences/extensionMethod/Extension.java‎

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
package pkg;
2+
3+
public static class Extension {
4+
public static String test(String s) {
5+
return s;
6+
}
7+
8+
public static String test(String s, int i) {
9+
return s;
10+
}
11+
12+
public static String test(String s, String... s2) {
13+
return s;
14+
}
15+
}

‎test/eclipse/resource/findreferences/extensionMethod/Usage.java‎

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
package pkg;
2+
3+
import lombok.experimental.ExtensionMethod;
4+
5+
@ExtensionMethod(Extension.class)
6+
public class Usage {
7+
public void test() {
8+
private String string;
9+
string.test();
10+
string.test("a");
11+
string.test(1);
12+
}
13+
}

‎test/eclipse/resource/rename/extensionMethod/after/Extension.java‎

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
package pkg;
2+
3+
public static class Extension {
4+
public static String newTest(String s) {
5+
return s;
6+
}
7+
8+
public static String test(String s, int i) {
9+
return s;
10+
}
11+
12+
public static String test(String s, String... s2) {
13+
return s;
14+
}
15+
}

‎test/eclipse/resource/rename/extensionMethod/after/Usage.java‎

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
package pkg;
2+
3+
import lombok.experimental.ExtensionMethod;
4+
5+
@ExtensionMethod(Extension.class)
6+
public class Usage {
7+
public void test() {
8+
private String string;
9+
string.newTest();
10+
string.test("a");
11+
string.test(1);
12+
}
13+
}

‎test/eclipse/resource/rename/extensionMethod/before/Extension.java‎

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
package pkg;
2+
3+
public static class Extension {
4+
public static String test(String s) {
5+
return s;
6+
}
7+
8+
public static String test(String s, int i) {
9+
return s;
10+
}
11+
12+
public static String test(String s, String... s2) {
13+
return s;
14+
}
15+
}

‎test/eclipse/resource/rename/extensionMethod/before/Usage.java‎

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
package pkg;
2+
3+
import lombok.experimental.ExtensionMethod;
4+
5+
@ExtensionMethod(Extension.class)
6+
public class Usage {
7+
public void test() {
8+
private String string;
9+
string.test();
10+
string.test("a");
11+
string.test(1);
12+
}
13+
}

‎test/eclipse/src/lombok/eclipse/EclipseTests.java‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,10 @@
88
import lombok.eclipse.edit.SelectTest;
99
import lombok.eclipse.refactoring.ExtractInterfaceTest;
1010
import lombok.eclipse.refactoring.RenameTest;
11+
import lombok.eclipse.references.FindReferencesTest;
1112

1213
@RunWith(Suite.class)
13-
@SuiteClasses({ExtractInterfaceTest.class, RenameTest.class, SelectTest.class, CleanupTest.class})
14+
@SuiteClasses({ExtractInterfaceTest.class, RenameTest.class, SelectTest.class, CleanupTest.class, FindReferencesTest.class})
1415
public class EclipseTests {
1516

1617
}

0 commit comments

Comments
 (0)