diff --git a/java/java-analysis-impl/src/com/intellij/codeInspection/canBeFinal/CanBeFinalInspection.java b/java/java-analysis-impl/src/com/intellij/codeInspection/canBeFinal/CanBeFinalInspection.java index 80109a54f4a7..b4aa6c0f0f11 100644 --- a/java/java-analysis-impl/src/com/intellij/codeInspection/canBeFinal/CanBeFinalInspection.java +++ b/java/java-analysis-impl/src/com/intellij/codeInspection/canBeFinal/CanBeFinalInspection.java @@ -161,6 +161,7 @@ public class CanBeFinalInspection extends GlobalJavaBatchInspectionTool { psiIdentifier = ((PsiMethod)psiMember).getNameIdentifier(); } else if (refElement instanceof RefField) { + if (!((RefField)refElement).isUsedForWriting()) return null; if (!isReportFields()) return null; psiIdentifier = ((PsiField)psiMember).getNameIdentifier(); } diff --git a/java/java-tests/testData/inspection/canBeFinal/fieldImplicitWrite/expected.xml b/java/java-tests/testData/inspection/canBeFinal/fieldImplicitWrite/expected.xml index 93b56fb7e0e4..4704d91e891d 100644 --- a/java/java-tests/testData/inspection/canBeFinal/fieldImplicitWrite/expected.xml +++ b/java/java-tests/testData/inspection/canBeFinal/fieldImplicitWrite/expected.xml @@ -1,9 +1,2 @@ - - - Test.java - 3 - Declaration can have final modifier - - - \ No newline at end of file + \ No newline at end of file