Skip to content

Commit 80268d9

Browse files
committed
Replacing EP_NAME with getExtensionPoint (part 4).
1 parent fc78153 commit 80268d9

3 files changed

Lines changed: 34 additions & 23 deletions

File tree

java-analysis-impl/src/main/java/com/intellij/java/analysis/impl/codeInsight/daemon/impl/UnusedSymbolUtil.java

Lines changed: 24 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
import com.intellij.java.language.impl.psi.impl.source.PsiClassImpl;
2424
import com.intellij.java.language.psi.*;
2525
import com.intellij.java.language.psi.util.PropertyUtil;
26+
import consulo.annotation.DeprecationInfo;
2627
import consulo.annotation.access.RequiredReadAction;
2728
import consulo.application.progress.ProgressIndicator;
2829
import consulo.application.progress.ProgressManager;
@@ -40,6 +41,7 @@
4041
import consulo.language.psi.resolve.RefResolveService;
4142
import consulo.language.psi.scope.GlobalSearchScope;
4243
import consulo.language.psi.search.PsiSearchHelper;
44+
import consulo.localize.LocalizeValue;
4345
import consulo.project.Project;
4446
import consulo.usage.UsageInfo;
4547
import jakarta.annotation.Nonnull;
@@ -87,28 +89,37 @@ public static boolean isImplicitWrite(@Nonnull Project project, @Nonnull PsiVari
8789
}) || isInjected(project, element);
8890
}
8991

90-
@Nullable
92+
@Nonnull
9193
@RequiredReadAction
92-
public static HighlightInfo createUnusedSymbolInfo(
94+
public static HighlightInfo.Builder createUnusedSymbolInfo(
9395
@Nonnull PsiElement element,
94-
@Nonnull String message,
96+
@Nonnull LocalizeValue message,
9597
@Nonnull HighlightInfoType highlightInfoType
9698
) {
97-
HighlightInfo info = HighlightInfo.newHighlightInfo(highlightInfoType)
99+
HighlightInfo.Builder hlBuilder = HighlightInfo.newHighlightInfo(highlightInfoType)
98100
.range(element)
99-
.descriptionAndTooltip(message).create();
100-
101-
if (info == null) {
102-
return null; //filtered out
103-
}
101+
.descriptionAndTooltip(message);
104102

105-
for (UnusedDeclarationFixProvider provider : UnusedDeclarationFixProvider.EP_NAME.getExtensionList()) {
103+
element.getApplication().getExtensionPoint(UnusedDeclarationFixProvider.class).forEach(provider -> {
106104
IntentionAction[] fixes = provider.getQuickFixes(element);
107105
for (IntentionAction fix : fixes) {
108-
info.registerFix(fix, null, null, null, null);
106+
hlBuilder.registerFix(fix);
109107
}
110-
}
111-
return info;
108+
});
109+
110+
return hlBuilder;
111+
}
112+
113+
@Deprecated
114+
@DeprecationInfo("Use variant with LocalizeValue")
115+
@Nullable
116+
@RequiredReadAction
117+
public static HighlightInfo createUnusedSymbolInfo(
118+
@Nonnull PsiElement element,
119+
@Nonnull String message,
120+
@Nonnull HighlightInfoType highlightInfoType
121+
) {
122+
return createUnusedSymbolInfo(element, LocalizeValue.of(message), highlightInfoType).create();
112123
}
113124

114125
@RequiredReadAction

plugin/src/main/java/com/intellij/java/impl/ig/performance/ClassInitializerMayBeStaticInspection.java

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,9 @@
2626
import com.siyeh.ig.psiutils.ClassUtils;
2727
import com.siyeh.localize.InspectionGadgetsLocalize;
2828
import consulo.annotation.component.ExtensionImpl;
29+
import consulo.application.Application;
30+
import consulo.component.extension.ExtensionPoint;
2931
import consulo.java.analysis.codeInspection.CantBeStaticCondition;
30-
import consulo.java.analysis.codeInspection.JavaExtensionPoints;
3132
import consulo.language.psi.PsiElement;
3233
import jakarta.annotation.Nonnull;
3334
import org.jetbrains.annotations.Nls;
@@ -76,11 +77,11 @@ public void visitClassInitializer(PsiClassInitializer initializer) {
7677
if (containingClass == null) {
7778
return;
7879
}
79-
for (CantBeStaticCondition addin : JavaExtensionPoints.CANT_BE_STATIC_EP_NAME.getExtensions()) {
80-
if (addin.cantBeStatic(initializer)) {
81-
return;
82-
}
80+
ExtensionPoint<CantBeStaticCondition> cantBeStaticEP = Application.get().getExtensionPoint(CantBeStaticCondition.class);
81+
if (cantBeStaticEP.anyMatchSafe(addin -> addin.cantBeStatic(initializer))) {
82+
return;
8383
}
84+
8485
PsiElement scope = containingClass.getScope();
8586
if (!(scope instanceof PsiJavaFile) && !containingClass.isStatic()) {
8687
return;

plugin/src/main/java/com/intellij/java/impl/ig/performance/MethodMayBeStaticInspection.java

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,9 @@
2727
import consulo.annotation.access.RequiredReadAction;
2828
import consulo.annotation.component.ExtensionImpl;
2929
import consulo.application.util.query.Query;
30+
import consulo.component.extension.ExtensionPoint;
3031
import consulo.deadCodeNotWorking.impl.MultipleCheckboxOptionsPanel;
3132
import consulo.java.analysis.codeInspection.CantBeStaticCondition;
32-
import consulo.java.analysis.codeInspection.JavaExtensionPoints;
3333
import consulo.language.psi.PsiElement;
3434
import consulo.util.lang.ref.SimpleReference;
3535
import jakarta.annotation.Nonnull;
@@ -98,10 +98,9 @@ public void visitMethod(@Nonnull PsiMethod method) {
9898
if (containingClass == null) {
9999
return;
100100
}
101-
for (CantBeStaticCondition addin : JavaExtensionPoints.CANT_BE_STATIC_EP_NAME.getExtensions()) {
102-
if (addin.cantBeStatic(method)) {
103-
return;
104-
}
101+
ExtensionPoint<CantBeStaticCondition> cantBeStaticEP = method.getApplication().getExtensionPoint(CantBeStaticCondition.class);
102+
if (cantBeStaticEP.anyMatchSafe(addin -> addin.cantBeStatic(method))) {
103+
return;
105104
}
106105
PsiElement scope = containingClass.getScope();
107106
if (!(scope instanceof PsiJavaFile) && !containingClass.isStatic()) {

0 commit comments

Comments
 (0)