azalea/kotlin-fork
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Refactoring (same logic in ControlFlowInformationProvider / DataFlowValueFactory)

Browse Source
This commit is contained in:
Mikhail Glukhikh
2016-03-23 13:18:35 +03:00
parent 32e7f9e58f
commit 46290798ca
2 changed files with 7 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files
compiler/frontend/src/org/jetbrains/kotlin/cfg/ControlFlowInformationProvider.java
+1 -1
View File
@@ -393,7 +393,7 @@ public class ControlFlowInformationProvider {
// Should return KtDeclarationWithBody or KtClassOrObject
@Nullable
private static KtDeclaration getElementParentDeclaration(@NotNull KtElement element) {
public static KtDeclaration getElementParentDeclaration(@NotNull KtElement element) {
//noinspection unchecked
return PsiTreeUtil.getParentOfType(element, KtDeclarationWithBody.class, KtClassOrObject.class);
}
compiler/frontend/src/org/jetbrains/kotlin/resolve/calls/smartcasts/DataFlowValueFactory.java
+6 -15
View File
@@ -22,6 +22,7 @@ import com.intellij.psi.tree.IElementType;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;
import org.jetbrains.kotlin.KtNodeTypes;
import org.jetbrains.kotlin.cfg.ControlFlowInformationProvider;
import org.jetbrains.kotlin.descriptors.*;
import org.jetbrains.kotlin.descriptors.impl.LocalVariableDescriptor;
import org.jetbrains.kotlin.descriptors.impl.SyntheticFieldDescriptor;
@@ -355,21 +356,11 @@ public class DataFlowValueFactory {
@NotNull BindingContext bindingContext,
@NotNull KtElement accessElement
) {
PsiElement parent = accessElement.getParent();
while (parent != null) {
// We are inside some declaration
if (parent instanceof KtDeclarationWithBody || parent instanceof KtClassOrObject) {
DeclarationDescriptor descriptor = bindingContext.get(DECLARATION_TO_DESCRIPTOR, parent);
if (variableContainingDeclaration.equals(descriptor)) {
// Access is at the same declaration: not in closure
break;
}
else {
// Access is lower than parent: in closure
return true;
}
}
parent = parent.getParent();
KtDeclaration parent = ControlFlowInformationProvider.getElementParentDeclaration(accessElement);
if (parent != null) {
DeclarationDescriptor descriptor = bindingContext.get(DECLARATION_TO_DESCRIPTOR, parent);
// Access is at the same declaration: not in closure, lower: in closure
return !variableContainingDeclaration.equals(descriptor);
}
return false;
}