MDS/openide
1
0
Fork 0
mirror of https://gitflic.ru/project/openide/openide.git synced 2026-01-08 15:09:39 +07:00
Code Issues Packages Projects Releases Wiki Activity

bring into scope: multiple vars in one declaration should be normalized (IDEA-62848 )

Browse Source
This commit is contained in:
anna
2010-12-13 15:55:43 +03:00
parent e68fb6bb8e
commit 64e3456f20
3 changed files with 21 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/BringVariableIntoScopeFix.java
View File

@@ -97,7 +97,7 @@ public class BringVariableIntoScopeFix implements IntentionAction {
public void invoke(@NotNull Project project, Editor editor, PsiFile file) throws IncorrectOperationException {
LOG.assertTrue(myOutOfScopeVariable != null);
PsiManager manager = file.getManager();
myOutOfScopeVariable.normalizeDeclaration();
PsiUtil.setModifierProperty(myOutOfScopeVariable, PsiModifier.FINAL, false);
PsiElement commonParent = PsiTreeUtil.findCommonParent(myOutOfScopeVariable, myUnresolvedReference);
LOG.assertTrue(commonParent != null);

11
java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/bringVariableIntoScope/afterMultipleDeclarations.java Normal file
View File

@@ -0,0 +1,11 @@
// "Bring 'int j' into Scope" "true"
class a {
void foo() {
int j;
{
int i;
j = 10;
}
System.out.println(<caret>j); // invoke "bring 'int j' into scope" quickfix here
}
}

9
java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/bringVariableIntoScope/beforeMultipleDeclarations.java Normal file
View File

@@ -0,0 +1,9 @@
// "Bring 'int j' into Scope" "true"
class a {
void foo() {
{
int i, j = 10;
}
System.out.println(<caret>j); // invoke "bring 'int j' into scope" quickfix here
}
}