MDS/openide
1
0
Fork 0
mirror of https://gitflic.ru/project/openide/openide.git synced 2026-05-06 05:10:22 +07:00
Code Issues Packages Projects Releases Wiki Activity

don't override first field by same named second (IDEA-174523)

Browse Source
This commit is contained in:
Anna Kozlova
2017-06-16 11:51:12 +03:00
parent c9f38674c2
commit 5d8e3f67a1
3 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
java/java-psi-impl/src/com/intellij/psi/impl/source/ClassInnerStuffCache.java
View File

@@ -149,7 +149,7 @@ public class ClassInnerStuffCache {
Map<String, PsiField> cachedFields = new THashMap<>();
for (PsiField field : fields) {
String name = field.getName();
if (!(field instanceof ExternallyDefinedPsiElement) || !cachedFields.containsKey(name)) {
if (!cachedFields.containsKey(name)) {
cachedFields.put(name, field);
}
}

4
java/java-tests/testData/codeInsight/daemonCodeAnalyzer/advHighlighting6/SameFieldNameErrorOnSecond.java Normal file
View File

@@ -0,0 +1,4 @@
class Test {
int i;
int <error descr="Variable 'i' is already defined in the scope">i</error>;
}

3
java/java-tests/testSrc/com/intellij/java/codeInsight/daemon/LightAdvHighlightingJdk6Test.java
View File

@@ -80,6 +80,9 @@ public class LightAdvHighlightingJdk6Test extends LightDaemonAnalyzerTestCase {
public void testThisInArgListOfAnonymous() {
doTest(false, false);
}
public void testSameFieldNameErrorOnSecond() {
doTest(false, false);
}
public void testEnumConstantWithoutInterfaceImplementation() throws Exception {
doTest(false, false);