removed method 'isClassObjectAValue'
There are no sensible usages of it. Presence of class object can be determined by 'getClassObjectDescriptor() != null' check
This commit is contained in:
@@ -30,6 +30,4 @@ public interface ClassifierDescriptor extends DeclarationDescriptorNonRoot {
|
|||||||
|
|
||||||
@Nullable
|
@Nullable
|
||||||
JetType getClassObjectType();
|
JetType getClassObjectType();
|
||||||
|
|
||||||
boolean isClassObjectAValue();
|
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -141,11 +141,6 @@ public class ClassDescriptorImpl extends DeclarationDescriptorNonRootImpl implem
|
|||||||
return kind;
|
return kind;
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
|
||||||
public boolean isClassObjectAValue() {
|
|
||||||
return true;
|
|
||||||
}
|
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public <R, D> R accept(DeclarationDescriptorVisitor<R, D> visitor, D data) {
|
public <R, D> R accept(DeclarationDescriptorVisitor<R, D> visitor, D data) {
|
||||||
return visitor.visitClassDescriptor(this, data);
|
return visitor.visitClassDescriptor(this, data);
|
||||||
|
|||||||
-5
@@ -180,11 +180,6 @@ public class LazySubstitutingClassDescriptor implements ClassDescriptor {
|
|||||||
return original.isInner();
|
return original.isInner();
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
|
||||||
public boolean isClassObjectAValue() {
|
|
||||||
return original.isClassObjectAValue();
|
|
||||||
}
|
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public <R, D> R accept(DeclarationDescriptorVisitor<R, D> visitor, D data) {
|
public <R, D> R accept(DeclarationDescriptorVisitor<R, D> visitor, D data) {
|
||||||
return visitor.visitClassDescriptor(this, data);
|
return visitor.visitClassDescriptor(this, data);
|
||||||
|
|||||||
-6
@@ -133,12 +133,6 @@ public abstract class MutableClassDescriptorLite extends ClassDescriptorBase {
|
|||||||
return classObjectType;
|
return classObjectType;
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
|
||||||
public boolean isClassObjectAValue() {
|
|
||||||
return true;
|
|
||||||
}
|
|
||||||
|
|
||||||
|
|
||||||
@NotNull
|
@NotNull
|
||||||
@Override
|
@Override
|
||||||
public ClassKind getKind() {
|
public ClassKind getKind() {
|
||||||
|
|||||||
-5
@@ -245,11 +245,6 @@ public class TypeParameterDescriptorImpl extends DeclarationDescriptorNonRootImp
|
|||||||
return classObjectBoundsAsType;
|
return classObjectBoundsAsType;
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
|
||||||
public boolean isClassObjectAValue() {
|
|
||||||
return true;
|
|
||||||
}
|
|
||||||
|
|
||||||
public void addClassObjectBound(@NotNull JetType bound) {
|
public void addClassObjectBound(@NotNull JetType bound) {
|
||||||
checkUninitialized();
|
checkUninitialized();
|
||||||
classObjectUpperBounds.add(bound); // TODO : Duplicates?
|
classObjectUpperBounds.add(bound); // TODO : Duplicates?
|
||||||
|
|||||||
+3
-10
@@ -72,9 +72,6 @@ public class CallExpressionResolver {
|
|||||||
if (classObjectType != null) {
|
if (classObjectType != null) {
|
||||||
context.trace.record(REFERENCE_TARGET, expression, classifier);
|
context.trace.record(REFERENCE_TARGET, expression, classifier);
|
||||||
JetType result = getExtendedClassObjectType(classObjectType, referencedName, classifier, context);
|
JetType result = getExtendedClassObjectType(classObjectType, referencedName, classifier, context);
|
||||||
if (result == null) {
|
|
||||||
context.trace.report(NO_CLASS_OBJECT.on(expression, classifier));
|
|
||||||
}
|
|
||||||
return DataFlowUtils.checkType(result, expression, context);
|
return DataFlowUtils.checkType(result, expression, context);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -120,7 +117,7 @@ public class CallExpressionResolver {
|
|||||||
return result[0];
|
return result[0];
|
||||||
}
|
}
|
||||||
|
|
||||||
@Nullable
|
@NotNull
|
||||||
private JetType getExtendedClassObjectType(
|
private JetType getExtendedClassObjectType(
|
||||||
@NotNull JetType classObjectType,
|
@NotNull JetType classObjectType,
|
||||||
@NotNull Name referencedName,
|
@NotNull Name referencedName,
|
||||||
@@ -135,18 +132,14 @@ public class CallExpressionResolver {
|
|||||||
|
|
||||||
NamespaceDescriptor namespace = context.scope.getNamespace(referencedName);
|
NamespaceDescriptor namespace = context.scope.getNamespace(referencedName);
|
||||||
if (namespace != null) {
|
if (namespace != null) {
|
||||||
|
//for enums loaded from java binaries
|
||||||
scopes.add(namespace.getMemberScope());
|
scopes.add(namespace.getMemberScope());
|
||||||
}
|
}
|
||||||
|
|
||||||
JetScope scope = new ChainedScope(classifier, scopes.toArray(new JetScope[scopes.size()]));
|
JetScope scope = new ChainedScope(classifier, scopes.toArray(new JetScope[scopes.size()]));
|
||||||
return new NamespaceType(referencedName, scope);
|
return new NamespaceType(referencedName, scope);
|
||||||
}
|
}
|
||||||
else if (context.expressionPosition == ExpressionPosition.LHS_OF_DOT || classifier.isClassObjectAValue()) {
|
return classObjectType;
|
||||||
return classObjectType;
|
|
||||||
}
|
|
||||||
else {
|
|
||||||
return null;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
private boolean furtherNameLookup(
|
private boolean furtherNameLookup(
|
||||||
|
|||||||
+7
-2
@@ -119,7 +119,12 @@ public class DataFlowValueFactory {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
private static final IdentifierInfo NO_IDENTIFIER_INFO = new IdentifierInfo(null, false, false);
|
private static final IdentifierInfo NO_IDENTIFIER_INFO = new IdentifierInfo(null, false, false) {
|
||||||
|
@Override
|
||||||
|
public String toString() {
|
||||||
|
return "NO_IDENTIFIER_INFO";
|
||||||
|
}
|
||||||
|
};
|
||||||
|
|
||||||
@NotNull
|
@NotNull
|
||||||
private static IdentifierInfo createInfo(Object id, boolean isStable) {
|
private static IdentifierInfo createInfo(Object id, boolean isStable) {
|
||||||
@@ -199,7 +204,7 @@ public class DataFlowValueFactory {
|
|||||||
}
|
}
|
||||||
if (declarationDescriptor instanceof ClassDescriptor) {
|
if (declarationDescriptor instanceof ClassDescriptor) {
|
||||||
ClassDescriptor classDescriptor = (ClassDescriptor) declarationDescriptor;
|
ClassDescriptor classDescriptor = (ClassDescriptor) declarationDescriptor;
|
||||||
return createInfo(classDescriptor, classDescriptor.isClassObjectAValue());
|
return createInfo(classDescriptor, classDescriptor.getClassObjectDescriptor() != null);
|
||||||
}
|
}
|
||||||
return NO_IDENTIFIER_INFO;
|
return NO_IDENTIFIER_INFO;
|
||||||
}
|
}
|
||||||
|
|||||||
-6
@@ -275,11 +275,6 @@ public class LazyClassDescriptor extends ClassDescriptorBase implements LazyDesc
|
|||||||
return classObjectDescriptor == null ? null : classObjectDescriptor.getDefaultType();
|
return classObjectDescriptor == null ? null : classObjectDescriptor.getDefaultType();
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
|
||||||
public boolean isClassObjectAValue() {
|
|
||||||
return true;
|
|
||||||
}
|
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public ClassDescriptor getClassObjectDescriptor() {
|
public ClassDescriptor getClassObjectDescriptor() {
|
||||||
return classObjectDescriptor.compute();
|
return classObjectDescriptor.compute();
|
||||||
@@ -395,7 +390,6 @@ public class LazyClassDescriptor extends ClassDescriptorBase implements LazyDesc
|
|||||||
ForceResolveUtil.forceResolveAllContents(getTypeConstructor());
|
ForceResolveUtil.forceResolveAllContents(getTypeConstructor());
|
||||||
getUnsubstitutedPrimaryConstructor();
|
getUnsubstitutedPrimaryConstructor();
|
||||||
getVisibility();
|
getVisibility();
|
||||||
isClassObjectAValue();
|
|
||||||
}
|
}
|
||||||
|
|
||||||
private class LazyClassTypeConstructor implements LazyDescriptor, TypeConstructor {
|
private class LazyClassTypeConstructor implements LazyDescriptor, TypeConstructor {
|
||||||
|
|||||||
-5
@@ -256,11 +256,6 @@ public class LazyTypeParameterDescriptor implements TypeParameterDescriptor, Laz
|
|||||||
return null;
|
return null;
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
|
||||||
public boolean isClassObjectAValue() {
|
|
||||||
return false;
|
|
||||||
}
|
|
||||||
|
|
||||||
@NotNull
|
@NotNull
|
||||||
@Override
|
@Override
|
||||||
public DeclarationDescriptor getOriginal() {
|
public DeclarationDescriptor getOriginal() {
|
||||||
|
|||||||
Reference in New Issue
Block a user