field can be local: don't evaluate runtime constants (IDEA-157179)

java/java-analysis-impl/src/com/intellij/codeInspection/varScopeCanBeNarrowed/FieldCanBeLocalInspectionBase.java

@@ -37,7 +37,6 @@ import com.intellij.psi.search.LocalSearchScope;
 import com.intellij.psi.search.searches.ReferencesSearch;
 import com.intellij.psi.util.PsiTreeUtil;
 import com.intellij.psi.util.PsiUtil;
-import com.intellij.util.Processor;
 import com.siyeh.InspectionGadgetsBundle;
 import gnu.trove.THashSet;
 import org.jdom.Element;
@@ -214,8 +213,8 @@ public class FieldCanBeLocalInspectionBase extends BaseJavaBatchLocalInspectionT
                                      Set<PsiField> ignored) {
     try {
       final Ref<Collection<PsiVariable>> writtenVariables = new Ref<Collection<PsiVariable>>();
-      final ControlFlow
-        controlFlow = ControlFlowFactory.getInstance(body.getProject()).getControlFlow(body, AllVariablesControlFlowPolicy.getInstance());
+      final ControlFlow controlFlow = ControlFlowFactory.getInstance(body.getProject())
+          .getControlFlow(body, AllVariablesControlFlowPolicy.getInstance(), false, false);
       final List<PsiVariable> usedVars = ControlFlowUtil.getUsedVariables(controlFlow, 0, controlFlow.getSize());
       for (PsiVariable usedVariable : usedVars) {
         if (usedVariable instanceof PsiField) {

java/java-tests/testData/inspection/fieldCanBeLocal/dontSimplifyRuntimeConstants/expected.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<problems/>

java/java-tests/testData/inspection/fieldCanBeLocal/dontSimplifyRuntimeConstants/src/Test.java

@@ -0,0 +1,15 @@
+class test {
+    private static final boolean DEBUG = false;
+
+    private test field;
+
+    private test(test field) {
+        this.field = field;
+    }
+
+    private void method() {
+        if (DEBUG) {
+            field.method();
+        }
+    }
+}

java/java-tests/testSrc/com/intellij/codeInspection/FieldCanBeLocalTest.java

@@ -41,6 +41,7 @@ public class FieldCanBeLocalTest extends InspectionTestCase {
   public void testFieldUsedForWritingInLambda() throws Exception {doTest();}
   public void testStaticQualifiedFieldAccessForWriting() throws Exception {doTest();}
   public void testFieldReferencedFromAnotherObject() throws Exception {doTest();}
+  public void testDontSimplifyRuntimeConstants() throws Exception {doTest();}
   public void testIgnoreAnnotated() throws Exception {
     final FieldCanBeLocalInspection inspection = new FieldCanBeLocalInspection();
     doTestConfigured(inspection);