diff options
Diffstat (limited to 'src/main/java/cuchaz/enigma/analysis/EntryReference.java')
| -rw-r--r-- | src/main/java/cuchaz/enigma/analysis/EntryReference.java | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/main/java/cuchaz/enigma/analysis/EntryReference.java b/src/main/java/cuchaz/enigma/analysis/EntryReference.java index 3761fca..b972585 100644 --- a/src/main/java/cuchaz/enigma/analysis/EntryReference.java +++ b/src/main/java/cuchaz/enigma/analysis/EntryReference.java | |||
| @@ -12,8 +12,8 @@ | |||
| 12 | package cuchaz.enigma.analysis; | 12 | package cuchaz.enigma.analysis; |
| 13 | 13 | ||
| 14 | import cuchaz.enigma.mapping.ClassEntry; | 14 | import cuchaz.enigma.mapping.ClassEntry; |
| 15 | import cuchaz.enigma.mapping.ConstructorEntry; | ||
| 16 | import cuchaz.enigma.mapping.Entry; | 15 | import cuchaz.enigma.mapping.Entry; |
| 16 | import cuchaz.enigma.mapping.MethodEntry; | ||
| 17 | import cuchaz.enigma.utils.Utils; | 17 | import cuchaz.enigma.utils.Utils; |
| 18 | 18 | ||
| 19 | import java.util.Arrays; | 19 | import java.util.Arrays; |
| @@ -21,7 +21,7 @@ import java.util.List; | |||
| 21 | 21 | ||
| 22 | public class EntryReference<E extends Entry, C extends Entry> { | 22 | public class EntryReference<E extends Entry, C extends Entry> { |
| 23 | 23 | ||
| 24 | private static final List<String> ConstructorNonNames = Arrays.asList("this", "super", "static"); | 24 | private static final List<String> CONSTRUCTOR_NON_NAMES = Arrays.asList("this", "super", "static"); |
| 25 | public E entry; | 25 | public E entry; |
| 26 | public C context; | 26 | public C context; |
| 27 | 27 | ||
| @@ -40,7 +40,7 @@ public class EntryReference<E extends Entry, C extends Entry> { | |||
| 40 | this.context = context; | 40 | this.context = context; |
| 41 | 41 | ||
| 42 | this.sourceName = sourceName != null && !sourceName.isEmpty(); | 42 | this.sourceName = sourceName != null && !sourceName.isEmpty(); |
| 43 | if (entry instanceof ConstructorEntry && ConstructorNonNames.contains(sourceName)) { | 43 | if (entry instanceof MethodEntry && ((MethodEntry) entry).isConstructor() && CONSTRUCTOR_NON_NAMES.contains(sourceName)) { |
| 44 | this.sourceName = false; | 44 | this.sourceName = false; |
| 45 | } | 45 | } |
| 46 | } | 46 | } |
| @@ -53,9 +53,9 @@ public class EntryReference<E extends Entry, C extends Entry> { | |||
| 53 | 53 | ||
| 54 | public ClassEntry getLocationClassEntry() { | 54 | public ClassEntry getLocationClassEntry() { |
| 55 | if (context != null) { | 55 | if (context != null) { |
| 56 | return context.getClassEntry(); | 56 | return context.getOwnerClassEntry(); |
| 57 | } | 57 | } |
| 58 | return entry.getClassEntry(); | 58 | return entry.getOwnerClassEntry(); |
| 59 | } | 59 | } |
| 60 | 60 | ||
| 61 | public boolean isNamed() { | 61 | public boolean isNamed() { |
| @@ -63,9 +63,9 @@ public class EntryReference<E extends Entry, C extends Entry> { | |||
| 63 | } | 63 | } |
| 64 | 64 | ||
| 65 | public Entry getNameableEntry() { | 65 | public Entry getNameableEntry() { |
| 66 | if (entry instanceof ConstructorEntry) { | 66 | if (entry instanceof MethodEntry && ((MethodEntry) entry).isConstructor()) { |
| 67 | // renaming a constructor really means renaming the class | 67 | // renaming a constructor really means renaming the class |
| 68 | return entry.getClassEntry(); | 68 | return entry.getOwnerClassEntry(); |
| 69 | } | 69 | } |
| 70 | return entry; | 70 | return entry; |
| 71 | } | 71 | } |