Rename: SOURCE_CODE_SHORT_NAMES_IN_TYPES -> *_SHORT_NAMES_NO_ANNOTATIONS

This commit is contained in:
Mikhail Glukhikh
2018-05-15 13:14:00 +03:00
parent 06f7e77006
commit 0f2879aa32
43 changed files with 65 additions and 66 deletions
@@ -168,7 +168,7 @@ public class QuickFixUtil {
FqName typeFqName = typeClassifierDescriptor != null ? DescriptorUtils.getFqNameSafe(typeClassifierDescriptor) : fqNameToCheckAgainst; FqName typeFqName = typeClassifierDescriptor != null ? DescriptorUtils.getFqNameSafe(typeClassifierDescriptor) : fqNameToCheckAgainst;
DescriptorRenderer renderer = typeFqName.shortName().equals(fqNameToCheckAgainst.shortName()) DescriptorRenderer renderer = typeFqName.shortName().equals(fqNameToCheckAgainst.shortName())
? IdeDescriptorRenderers.SOURCE_CODE ? IdeDescriptorRenderers.SOURCE_CODE
: IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES; : IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS;
return renderer.renderType(type); return renderer.renderType(type);
} }
} }
@@ -77,7 +77,7 @@ object IdeDescriptorRenderers {
typeNormalizer = { APPROXIMATE_FLEXIBLE_TYPES_NOT_NULL(unwrapAnonymousType(it)) } typeNormalizer = { APPROXIMATE_FLEXIBLE_TYPES_NOT_NULL(unwrapAnonymousType(it)) }
} }
@JvmField val SOURCE_CODE_SHORT_NAMES_IN_TYPES: DescriptorRenderer = BASE.withOptions { @JvmField val SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS: DescriptorRenderer = BASE.withOptions {
classifierNamePolicy = ClassifierNamePolicy.SHORT classifierNamePolicy = ClassifierNamePolicy.SHORT
typeNormalizer = { APPROXIMATE_FLEXIBLE_TYPES(unwrapAnonymousType(it)) } typeNormalizer = { APPROXIMATE_FLEXIBLE_TYPES(unwrapAnonymousType(it)) }
modifiers -= DescriptorRendererModifier.ANNOTATIONS modifiers -= DescriptorRendererModifier.ANNOTATIONS
@@ -300,7 +300,7 @@ fun BasicLookupElementFactory.createLookupElementForType(type: KotlinType): Look
if (type.isError) return null if (type.isError) return null
return if (type.isFunctionType) { return if (type.isFunctionType) {
val text = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(type) val text = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(type)
val baseLookupElement = LookupElementBuilder.create(text).withIcon(KotlinIcons.LAMBDA) val baseLookupElement = LookupElementBuilder.create(text).withIcon(KotlinIcons.LAMBDA)
BaseTypeLookupElement(type, baseLookupElement) BaseTypeLookupElement(type, baseLookupElement)
} }
@@ -308,7 +308,7 @@ fun BasicLookupElementFactory.createLookupElementForType(type: KotlinType): Look
val classifier = type.constructor.declarationDescriptor ?: return null val classifier = type.constructor.declarationDescriptor ?: return null
val baseLookupElement = createLookupElement(classifier, qualifyNestedClasses = true, includeClassTypeArguments = false) val baseLookupElement = createLookupElement(classifier, qualifyNestedClasses = true, includeClassTypeArguments = false)
val itemText = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(type) val itemText = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(type)
val typeLookupElement = object : BaseTypeLookupElement(type, baseLookupElement) { val typeLookupElement = object : BaseTypeLookupElement(type, baseLookupElement) {
override fun renderElement(presentation: LookupElementPresentation) { override fun renderElement(presentation: LookupElementPresentation) {
@@ -98,7 +98,7 @@ object LambdaSignatureTemplates {
} }
fun signaturePresentation(lambdaType: KotlinType, presentationKind: SignaturePresentation): String { fun signaturePresentation(lambdaType: KotlinType, presentationKind: SignaturePresentation): String {
fun typePresentation(type: KotlinType) = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(type) fun typePresentation(type: KotlinType) = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(type)
fun parameterPresentation(parameterType: KotlinType): String { fun parameterPresentation(parameterType: KotlinType): String {
val name = parameterType.extractParameterNameFromFunctionTypeArgument()?.render() val name = parameterType.extractParameterNameFromFunctionTypeArgument()?.render()
@@ -47,7 +47,7 @@ class OverridesCompletion(
private val collector: LookupElementsCollector, private val collector: LookupElementsCollector,
private val lookupElementFactory: BasicLookupElementFactory private val lookupElementFactory: BasicLookupElementFactory
) { ) {
private val PRESENTATION_RENDERER = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.withOptions { private val PRESENTATION_RENDERER = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.withOptions {
modifiers = emptySet() modifiers = emptySet()
includeAdditionalModifiers = false includeAdditionalModifiers = false
} }
@@ -177,7 +177,7 @@ class TypeInstantiationItems(
val allTypeArgsKnown = fuzzyType.freeParameters.isEmpty() || typeArgs.none { it.type.areTypeParametersUsedInside(fuzzyType.freeParameters) } val allTypeArgsKnown = fuzzyType.freeParameters.isEmpty() || typeArgs.none { it.type.areTypeParametersUsedInside(fuzzyType.freeParameters) }
itemText += if (allTypeArgsKnown) { itemText += if (allTypeArgsKnown) {
IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderTypeArguments(typeArgsToUse) IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderTypeArguments(typeArgsToUse)
} }
else { else {
"<...>" "<...>"
@@ -26,7 +26,7 @@ class KotlinMethodSmartStepTarget(
} }
companion object { companion object {
private val renderer = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.withOptions { private val renderer = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.withOptions {
parameterNameRenderingPolicy = ParameterNameRenderingPolicy.NONE parameterNameRenderingPolicy = ParameterNameRenderingPolicy.NONE
withoutReturnType = true withoutReturnType = true
renderAccessors = true renderAccessors = true
@@ -62,7 +62,7 @@ fun getPropertiesToUseInGeneratedMember(classOrObject: KtClassOrObject): List<Kt
} }
} }
private val MEMBER_RENDERER = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.withOptions { private val MEMBER_RENDERER = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.withOptions {
modifiers = emptySet() modifiers = emptySet()
startFromName = true startFromName = true
parameterNameRenderingPolicy = ParameterNameRenderingPolicy.NONE parameterNameRenderingPolicy = ParameterNameRenderingPolicy.NONE
@@ -72,7 +72,7 @@ class KotlinFindUsagesProvider : FindUsagesProvider {
is KtFunction -> { is KtFunction -> {
val name = element.name ?: "" val name = element.name ?: ""
val descriptor = element.unsafeResolveToDescriptor() as FunctionDescriptor val descriptor = element.unsafeResolveToDescriptor() as FunctionDescriptor
val renderer = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES val renderer = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS
val paramsDescription = descriptor.valueParameters.joinToString(prefix = "(", postfix = ")") { renderer.renderType(it.type) } val paramsDescription = descriptor.valueParameters.joinToString(prefix = "(", postfix = ")") { renderer.renderType(it.type) }
val returnType = descriptor.returnType val returnType = descriptor.returnType
val returnTypeDescription = if (returnType != null && !returnType.isUnit()) renderer.renderType(returnType) else null val returnTypeDescription = if (returnType != null && !returnType.isUnit()) renderer.renderType(returnType) else null
@@ -28,7 +28,7 @@ import java.util.*
import javax.swing.JList import javax.swing.JList
class KotlinSearchEverywherePsiRenderer(private val list: JList<*>) : DefaultPsiElementCellRenderer() { class KotlinSearchEverywherePsiRenderer(private val list: JList<*>) : DefaultPsiElementCellRenderer() {
private val RENDERER = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.withOptions { private val RENDERER = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.withOptions {
parameterNameRenderingPolicy = ParameterNameRenderingPolicy.NONE parameterNameRenderingPolicy = ParameterNameRenderingPolicy.NONE
modifiers = emptySet() modifiers = emptySet()
startFromName = false startFromName = false
@@ -247,7 +247,7 @@ open class ConvertLambdaToReferenceIntention(text: String) :
val originalReceiverType = receiverDescriptor.type val originalReceiverType = receiverDescriptor.type
val receiverType = originalReceiverType.approximateFlexibleTypes(preferNotNull = true) val receiverType = originalReceiverType.approximateFlexibleTypes(preferNotNull = true)
if (shortTypes) { if (shortTypes) {
"${IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(receiverType)}::$selectorReferenceName" "${IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(receiverType)}::$selectorReferenceName"
} }
else { else {
"${IdeDescriptorRenderers.SOURCE_CODE.renderType(receiverType)}::$selectorReferenceName" "${IdeDescriptorRenderers.SOURCE_CODE.renderType(receiverType)}::$selectorReferenceName"
@@ -41,7 +41,7 @@ class IterateExpressionIntention : SelfTargetingIntention<KtExpression>(KtExpres
val range = element.textRange val range = element.textRange
if (caretOffset != range.startOffset && caretOffset != range.endOffset) return false if (caretOffset != range.startOffset && caretOffset != range.endOffset) return false
val data = data(element) ?: return false val data = data(element) ?: return false
text = "Iterate over '${IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(data.collectionType)}'" text = "Iterate over '${IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(data.collectionType)}'"
return true return true
} }
@@ -171,8 +171,8 @@ class MoveMemberToCompanionObjectIntention : SelfTargetingRangeIntention<KtNamed
nameSuggestions = getNameSuggestionsForOuterInstance(element) nameSuggestions = getNameSuggestionsForOuterInstance(element)
val newParam = parameterList.addParameterBefore( val newParam = parameterList.addParameterBefore(
ktPsiFactory.createParameter("${nameSuggestions.first()}: ${IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(newParamType)}"), ktPsiFactory.createParameter("${nameSuggestions.first()}: ${IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(newParamType)}"),
parameters.firstOrNull() parameters.firstOrNull()
) )
val newOuterInstanceRef = ktPsiFactory.createExpression(newParam.name!!) val newOuterInstanceRef = ktPsiFactory.createExpression(newParam.name!!)
@@ -46,7 +46,7 @@ class ReconstructTypeInCastOrIsIntention : SelfTargetingOffsetIndependentIntenti
// No type parameters expected => nothing to reconstruct // No type parameters expected => nothing to reconstruct
if (type.constructor.parameters.isEmpty()) return false if (type.constructor.parameters.isEmpty()) return false
val typePresentation = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(type) val typePresentation = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(type)
text = "Replace by '$typePresentation'" text = "Replace by '$typePresentation'"
return true return true
@@ -182,7 +182,7 @@ class SpecifyTypeExplicitlyIntention : SelfTargetingRangeIntention<KtCallableDec
private class TypeChooseValueExpression( private class TypeChooseValueExpression(
items: List<KotlinType>, defaultItem: KotlinType items: List<KotlinType>, defaultItem: KotlinType
) : ChooseValueExpression<KotlinType>(items, defaultItem) { ) : ChooseValueExpression<KotlinType>(items, defaultItem) {
override fun getLookupString(element: KotlinType) = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(element) override fun getLookupString(element: KotlinType) = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(element)
override fun getResult(element: KotlinType) = IdeDescriptorRenderers.SOURCE_CODE.renderType(element) override fun getResult(element: KotlinType) = IdeDescriptorRenderers.SOURCE_CODE.renderType(element)
} }
@@ -151,7 +151,7 @@ class AddFunctionToSupertypeFix private constructor(
val targetClass = DescriptorToSourceUtilsIde.getAnyDeclaration(project, classDescriptor) as? KtClass ?: return null val targetClass = DescriptorToSourceUtilsIde.getAnyDeclaration(project, classDescriptor) as? KtClass ?: return null
return FunctionData( return FunctionData(
IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.render(functionDescriptor), IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.render(functionDescriptor),
sourceCode, sourceCode,
targetClass targetClass
) )
@@ -56,7 +56,7 @@ class AddTypeAnnotationToValueParameterFix(element: KtParameter) : KotlinQuickFi
} }
} }
typeNameShort = type?.let { IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(it) } typeNameShort = type?.let { IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(it) }
typeName = type?.let { IdeDescriptorRenderers.SOURCE_CODE.renderType(it) } typeName = type?.let { IdeDescriptorRenderers.SOURCE_CODE.renderType(it) }
} }
@@ -42,7 +42,7 @@ class ChangeAccessorTypeFix(element: KtPropertyAccessor) : KotlinQuickFixAction<
override fun getText(): String { override fun getText(): String {
val element = element ?: return "" val element = element ?: return ""
val type = getType() ?: return familyName val type = getType() ?: return familyName
val renderedType = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(type) val renderedType = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(type)
val target = if (element.isGetter) "getter" else "setter parameter" val target = if (element.isGetter) "getter" else "setter parameter"
return "Change $target type to $renderedType" return "Change $target type to $renderedType"
} }
@@ -34,7 +34,7 @@ import org.jetbrains.kotlin.resolve.lazy.BodyResolveMode
import org.jetbrains.kotlin.types.KotlinType import org.jetbrains.kotlin.types.KotlinType
class ChangeParameterTypeFix(element: KtParameter, type: KotlinType) : KotlinQuickFixAction<KtParameter>(element) { class ChangeParameterTypeFix(element: KtParameter, type: KotlinType) : KotlinQuickFixAction<KtParameter>(element) {
private val typePresentation = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(type) private val typePresentation = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(type)
private val typeInfo = KotlinTypeInfo(isCovariant = false, text = IdeDescriptorRenderers.SOURCE_CODE_NOT_NULL_TYPE_APPROXIMATION.renderType(type)) private val typeInfo = KotlinTypeInfo(isCovariant = false, text = IdeDescriptorRenderers.SOURCE_CODE_NOT_NULL_TYPE_APPROXIMATION.renderType(type))
private val containingDeclarationName: String? private val containingDeclarationName: String?
@@ -37,7 +37,7 @@ import org.jetbrains.kotlin.types.KotlinType
class ConvertExtensionToFunctionTypeFix(element: KtTypeReference, type: KotlinType) : KotlinQuickFixAction<KtTypeReference>(element) { class ConvertExtensionToFunctionTypeFix(element: KtTypeReference, type: KotlinType) : KotlinQuickFixAction<KtTypeReference>(element) {
private val targetTypeStringShort = type.renderType(IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES) private val targetTypeStringShort = type.renderType(IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS)
private val targetTypeStringLong = type.renderType(IdeDescriptorRenderers.SOURCE_CODE) private val targetTypeStringLong = type.renderType(IdeDescriptorRenderers.SOURCE_CODE)
override fun getText() = "Convert supertype to '$targetTypeStringShort'" override fun getText() = "Convert supertype to '$targetTypeStringShort'"
@@ -39,7 +39,7 @@ class LetImplementInterfaceFix(
expressionType: KotlinType expressionType: KotlinType
) : KotlinQuickFixAction<KtClassOrObject>(element), LowPriorityAction { ) : KotlinQuickFixAction<KtClassOrObject>(element), LowPriorityAction {
private fun KotlinType.renderShort() = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(this) private fun KotlinType.renderShort() = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(this)
private val expectedTypeName: String private val expectedTypeName: String
@@ -40,7 +40,7 @@ class NumberConversionFix(
val expressionType = element.analyze(BodyResolveMode.PARTIAL).getType(element) val expressionType = element.analyze(BodyResolveMode.PARTIAL).getType(element)
expressionType != null && expressionType != type && expressionType.isPrimitiveNumberType() && type.isPrimitiveNumberType() expressionType != null && expressionType != type && expressionType.isPrimitiveNumberType() && type.isPrimitiveNumberType()
} }
private val typePresentation = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(type) private val typePresentation = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(type)
override fun isAvailable(project: Project, editor: Editor?, file: KtFile) override fun isAvailable(project: Project, editor: Editor?, file: KtFile)
= disableIfAvailable?.isAvailable(project, editor, file) != true = disableIfAvailable?.isAvailable(project, editor, file) != true
@@ -23,7 +23,7 @@ import org.jetbrains.kotlin.diagnostics.Diagnostic
import org.jetbrains.kotlin.diagnostics.Errors.CAST_NEVER_SUCCEEDS import org.jetbrains.kotlin.diagnostics.Errors.CAST_NEVER_SUCCEEDS
import org.jetbrains.kotlin.idea.caches.resolve.analyze import org.jetbrains.kotlin.idea.caches.resolve.analyze
import org.jetbrains.kotlin.idea.core.replaced import org.jetbrains.kotlin.idea.core.replaced
import org.jetbrains.kotlin.idea.util.IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES import org.jetbrains.kotlin.idea.util.IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS
import org.jetbrains.kotlin.psi.* import org.jetbrains.kotlin.psi.*
import org.jetbrains.kotlin.psi.psiUtil.endOffset import org.jetbrains.kotlin.psi.psiUtil.endOffset
import org.jetbrains.kotlin.resolve.calls.callUtil.getType import org.jetbrains.kotlin.resolve.calls.callUtil.getType
@@ -59,7 +59,7 @@ class ReplacePrimitiveCastWithNumberConversionFix(
val castType = binaryExpression.right?.createTypeBinding(context)?.type ?: return null val castType = binaryExpression.right?.createTypeBinding(context)?.type ?: return null
if (!castType.isPrimitiveNumberType()) return null if (!castType.isPrimitiveNumberType()) return null
return ReplacePrimitiveCastWithNumberConversionFix(binaryExpression, SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(castType)) return ReplacePrimitiveCastWithNumberConversionFix(binaryExpression, SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(castType))
} }
} }
} }
@@ -94,7 +94,7 @@ private fun KotlinType.renderSingle(typeParameterNameMap: Map<TypeParameterDescr
.mapKeys { it.key.typeConstructor } .mapKeys { it.key.typeConstructor }
val typeToRender = TypeSubstitutor.create(substitution).substitute(this, Variance.INVARIANT)!! val typeToRender = TypeSubstitutor.create(substitution).substitute(this, Variance.INVARIANT)!!
val renderer = if (fq) IdeDescriptorRenderers.SOURCE_CODE else IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES val renderer = if (fq) IdeDescriptorRenderers.SOURCE_CODE else IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS
return renderer.renderType(typeToRender) return renderer.renderType(typeToRender)
} }
@@ -69,7 +69,7 @@ abstract class CreateCallableFromUsageFixBase<E : KtElement>(
private fun getDeclaration(descriptor: ClassifierDescriptor, project: Project): PsiElement? { private fun getDeclaration(descriptor: ClassifierDescriptor, project: Project): PsiElement? {
if (descriptor is FunctionClassDescriptor) { if (descriptor is FunctionClassDescriptor) {
val psiFactory = KtPsiFactory(project) val psiFactory = KtPsiFactory(project)
val syntheticClass = psiFactory.createClass(IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.render(descriptor)) val syntheticClass = psiFactory.createClass(IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.render(descriptor))
return psiFactory.createAnalyzableFile("${descriptor.name.asString()}.kt", "", element!!).add(syntheticClass) return psiFactory.createAnalyzableFile("${descriptor.name.asString()}.kt", "", element!!).add(syntheticClass)
} }
return DescriptorToSourceUtilsIde.getAnyDeclaration(project, descriptor) return DescriptorToSourceUtilsIde.getAnyDeclaration(project, descriptor)
@@ -114,12 +114,12 @@ abstract class CreateCallableFromUsageFixBase<E : KtElement>(
if (receiverType != null) { if (receiverType != null) {
if (isExtension) { if (isExtension) {
val receiverTypeText = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(receiverType) val receiverTypeText = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(receiverType)
val isFunctionType = receiverType.constructor.declarationDescriptor is FunctionClassDescriptor val isFunctionType = receiverType.constructor.declarationDescriptor is FunctionClassDescriptor
append(if (isFunctionType) "($receiverTypeText)" else receiverTypeText).append('.') append(if (isFunctionType) "($receiverTypeText)" else receiverTypeText).append('.')
} else { } else {
receiverType.constructor.declarationDescriptor?.let { receiverType.constructor.declarationDescriptor?.let {
append(IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderClassifierName(it)).append('.') append(IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderClassifierName(it)).append('.')
} }
} }
} }
@@ -64,7 +64,7 @@ abstract class CreateCallableFromUsageFixBase<E : KtElement>(
private fun getDeclaration(descriptor: ClassifierDescriptor, project: Project): PsiElement? { private fun getDeclaration(descriptor: ClassifierDescriptor, project: Project): PsiElement? {
if (descriptor is FunctionClassDescriptor) { if (descriptor is FunctionClassDescriptor) {
val psiFactory = KtPsiFactory(project) val psiFactory = KtPsiFactory(project)
val syntheticClass = psiFactory.createClass(IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.render(descriptor)) val syntheticClass = psiFactory.createClass(IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.render(descriptor))
return psiFactory.createAnalyzableFile("${descriptor.name.asString()}.kt", "", element!!).add(syntheticClass) return psiFactory.createAnalyzableFile("${descriptor.name.asString()}.kt", "", element!!).add(syntheticClass)
} }
return DescriptorToSourceUtilsIde.getAnyDeclaration(project, descriptor) return DescriptorToSourceUtilsIde.getAnyDeclaration(project, descriptor)
@@ -108,13 +108,13 @@ abstract class CreateCallableFromUsageFixBase<E : KtElement>(
if (receiverType != null) { if (receiverType != null) {
if (isExtension) { if (isExtension) {
val receiverTypeText = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(receiverType) val receiverTypeText = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(receiverType)
val isFunctionType = receiverType.constructor.declarationDescriptor is FunctionClassDescriptor val isFunctionType = receiverType.constructor.declarationDescriptor is FunctionClassDescriptor
append(if (isFunctionType) "($receiverTypeText)" else receiverTypeText).append('.') append(if (isFunctionType) "($receiverTypeText)" else receiverTypeText).append('.')
} }
else { else {
receiverType.constructor.declarationDescriptor?.let { receiverType.constructor.declarationDescriptor?.let {
append(IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderClassifierName(it)).append('.') append(IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderClassifierName(it)).append('.')
} }
} }
} }
@@ -60,7 +60,7 @@ class KotlinChangeSignatureData(
val jetParameter = valueParameters?.get(parameterDescriptor.index) val jetParameter = valueParameters?.get(parameterDescriptor.index)
val parameterType = parameterDescriptor.type val parameterType = parameterDescriptor.type
val parameterTypeText = jetParameter?.typeReference?.text val parameterTypeText = jetParameter?.typeReference?.text
?: IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(parameterType) ?: IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(parameterType)
KotlinParameterInfo( KotlinParameterInfo(
callableDescriptor = baseDescriptor, callableDescriptor = baseDescriptor,
originalIndex = parameterDescriptor.index, originalIndex = parameterDescriptor.index,
@@ -76,7 +76,7 @@ class KotlinChangeSignatureData(
val receiverType = baseDescriptor.extensionReceiverParameter?.type ?: return null val receiverType = baseDescriptor.extensionReceiverParameter?.type ?: return null
val receiverName = suggestReceiverNames(baseDeclaration.project, baseDescriptor).first() val receiverName = suggestReceiverNames(baseDeclaration.project, baseDescriptor).first()
val receiverTypeText = (baseDeclaration as? KtCallableDeclaration)?.receiverTypeReference?.text val receiverTypeText = (baseDeclaration as? KtCallableDeclaration)?.receiverTypeReference?.text
?: IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(receiverType) ?: IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(receiverType)
return KotlinParameterInfo(callableDescriptor = baseDescriptor, return KotlinParameterInfo(callableDescriptor = baseDescriptor,
name = receiverName, name = receiverName,
originalTypeInfo = KotlinTypeInfo(false, receiverType, receiverTypeText)) originalTypeInfo = KotlinTypeInfo(false, receiverType, receiverTypeText))
@@ -196,7 +196,7 @@ class KotlinChangeSignatureUsageProcessor : ChangeSignatureUsageProcessor {
result.add( result.add(
object : UnresolvableCollisionUsageInfo(callElement, null) { object : UnresolvableCollisionUsageInfo(callElement, null) {
override fun getDescription(): String { override fun getDescription(): String {
val signature = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.render(callerDescriptor) val signature = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.render(callerDescriptor)
return "There is already a variable '$currentName' in $signature. It will conflict with the new parameter." return "There is already a variable '$currentName' in $signature. It will conflict with the new parameter."
} }
} }
@@ -615,7 +615,7 @@ class KotlinChangeSignatureUsageProcessor : ChangeSignatureUsageProcessor {
callerDescriptor: DeclarationDescriptor) { callerDescriptor: DeclarationDescriptor) {
val valueParameters = caller.getValueParameters() val valueParameters = caller.getValueParameters()
val existingParameters = valueParameters.associateBy { it.name } val existingParameters = valueParameters.associateBy { it.name }
val signature = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.render(callerDescriptor) val signature = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.render(callerDescriptor)
for (parameterInfo in changeInfo.getNonReceiverParameters()) { for (parameterInfo in changeInfo.getNonReceiverParameters()) {
if (!(parameterInfo.isNewParameter)) continue if (!(parameterInfo.isNewParameter)) continue
@@ -58,7 +58,7 @@ val KotlinMethodDescriptor.returnTypeInfo: KotlinTypeInfo
get() { get() {
val type = baseDescriptor.returnType val type = baseDescriptor.returnType
val text = (baseDeclaration as? KtCallableDeclaration)?.typeReference?.text val text = (baseDeclaration as? KtCallableDeclaration)?.typeReference?.text
?: type?.let { IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(type) } ?: type?.let { IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(type) }
?: "Unit" ?: "Unit"
return KotlinTypeInfo(true, type, text) return KotlinTypeInfo(true, type, text)
} }
@@ -67,6 +67,6 @@ val KotlinMethodDescriptor.receiverTypeInfo: KotlinTypeInfo
get() { get() {
val type = baseDescriptor.extensionReceiverParameter?.type val type = baseDescriptor.extensionReceiverParameter?.type
val text = (baseDeclaration as? KtCallableDeclaration)?.receiverTypeReference?.text val text = (baseDeclaration as? KtCallableDeclaration)?.receiverTypeReference?.text
?: type?.let { IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(type) } ?: type?.let { IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(type) }
return KotlinTypeInfo(false, type, text) return KotlinTypeInfo(false, type, text)
} }
@@ -56,7 +56,7 @@ open class ExtractFunctionParameterTablePanel : AbstractParameterTablePanel<Para
override fun getTableCellRendererComponent( override fun getTableCellRendererComponent(
table: JTable, value: Any, isSelected: Boolean, hasFocus: Boolean, row: Int, column: Int table: JTable, value: Any, isSelected: Boolean, hasFocus: Boolean, row: Int, column: Int
): Component { ): Component {
myLabel.text = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(value as KotlinType) myLabel.text = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(value as KotlinType)
myLabel.background = if (isSelected) table.selectionBackground else table.background myLabel.background = if (isSelected) table.selectionBackground else table.background
myLabel.foreground = if (isSelected) table.selectionForeground else table.foreground myLabel.foreground = if (isSelected) table.selectionForeground else table.foreground
if (isSelected) { if (isSelected) {
@@ -80,7 +80,7 @@ open class ExtractFunctionParameterTablePanel : AbstractParameterTablePanel<Para
myEditorComponent.setCell(table, row, column) myEditorComponent.setCell(table, row, column)
myEditorComponent.setOptions(*info.originalParameter.getParameterTypeCandidates(false).toTypedArray()) myEditorComponent.setOptions(*info.originalParameter.getParameterTypeCandidates(false).toTypedArray())
myEditorComponent.setDefaultValue(info.type) myEditorComponent.setDefaultValue(info.type)
myEditorComponent.setToString { IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(it as KotlinType) } myEditorComponent.setToString { IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(it as KotlinType) }
return myEditorComponent return myEditorComponent
} }
@@ -114,7 +114,7 @@ public class KotlinExtractFunctionDialog extends DialogWrapper {
setOKActionEnabled(checkNames()); setOKActionEnabled(checkNames());
signaturePreviewField.setText( signaturePreviewField.setText(
ExtractorUtilKt.getSignaturePreview(getCurrentConfiguration(), IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES) ExtractorUtilKt.getSignaturePreview(getCurrentConfiguration(), IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS)
); );
} }
@@ -156,7 +156,7 @@ public class KotlinExtractFunctionDialog extends DialogWrapper {
boolean cellHasFocus boolean cellHasFocus
) { ) {
super.getListCellRendererComponent(list, value, index, isSelected, cellHasFocus); super.getListCellRendererComponent(list, value, index, isSelected, cellHasFocus);
setText(IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType((KotlinType) value)); setText(IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType((KotlinType) value));
return this; return this;
} }
} }
@@ -56,7 +56,6 @@ import org.jetbrains.kotlin.idea.util.IdeDescriptorRenderers
import org.jetbrains.kotlin.idea.util.approximateWithResolvableType import org.jetbrains.kotlin.idea.util.approximateWithResolvableType
import org.jetbrains.kotlin.idea.util.getResolutionScope import org.jetbrains.kotlin.idea.util.getResolutionScope
import org.jetbrains.kotlin.idea.util.isResolvableInScope import org.jetbrains.kotlin.idea.util.isResolvableInScope
import org.jetbrains.kotlin.lexer.KtModifierKeywordToken
import org.jetbrains.kotlin.lexer.KtTokens import org.jetbrains.kotlin.lexer.KtTokens
import org.jetbrains.kotlin.psi.* import org.jetbrains.kotlin.psi.*
import org.jetbrains.kotlin.psi.psiUtil.* import org.jetbrains.kotlin.psi.psiUtil.*
@@ -76,7 +75,7 @@ internal val KotlinBuiltIns.defaultReturnType: KotlinType get() = unitType
internal val KotlinBuiltIns.defaultParameterType: KotlinType get() = nullableAnyType internal val KotlinBuiltIns.defaultParameterType: KotlinType get() = nullableAnyType
private fun DeclarationDescriptor.renderForMessage(): String = private fun DeclarationDescriptor.renderForMessage(): String =
IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.render(this) IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.render(this)
private val TYPE_RENDERER = DescriptorRenderer.FQ_NAMES_IN_TYPES.withOptions { private val TYPE_RENDERER = DescriptorRenderer.FQ_NAMES_IN_TYPES.withOptions {
typeNormalizer = IdeDescriptorRenderers.APPROXIMATE_FLEXIBLE_TYPES typeNormalizer = IdeDescriptorRenderers.APPROXIMATE_FLEXIBLE_TYPES
@@ -80,7 +80,7 @@ class KotlinIntroduceParameterDialog private constructor(
) )
private val typeNameSuggestions = typeSuggestions private val typeNameSuggestions = typeSuggestions
.map { IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(it) } .map { IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(it) }
.toTypedArray() .toTypedArray()
private val nameField = NameSuggestionsField(nameSuggestions, project, KotlinFileType.INSTANCE) private val nameField = NameSuggestionsField(nameSuggestions, project, KotlinFileType.INSTANCE)
@@ -237,7 +237,7 @@ open class KotlinIntroduceParameterHandler(
if (expressionType.isUnit() || expressionType.isNothing()) { if (expressionType.isUnit() || expressionType.isNothing()) {
val message = KotlinRefactoringBundle.message( val message = KotlinRefactoringBundle.message(
"cannot.introduce.parameter.of.0.type", "cannot.introduce.parameter.of.0.type",
IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(expressionType) IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(expressionType)
) )
showErrorHint(project, editor, message, INTRODUCE_PARAMETER) showErrorHint(project, editor, message, INTRODUCE_PARAMETER)
return return
@@ -308,16 +308,16 @@ open class KotlinIntroduceParameterHandler(
val introduceParameterDescriptor = val introduceParameterDescriptor =
helper.configure( helper.configure(
IntroduceParameterDescriptor( IntroduceParameterDescriptor(
originalRange = originalExpression.toRange(), originalRange = originalExpression.toRange(),
callable = targetParent, callable = targetParent,
callableDescriptor = functionDescriptor, callableDescriptor = functionDescriptor,
newParameterName = suggestedNames.first().quoteIfNeeded(), newParameterName = suggestedNames.first().quoteIfNeeded(),
newParameterTypeText = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(replacementType), newParameterTypeText = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(replacementType),
argumentValue = originalExpression, argumentValue = originalExpression,
withDefaultValue = false, withDefaultValue = false,
parametersUsages = parametersUsages, parametersUsages = parametersUsages,
occurrencesToReplace = occurrencesToReplace, occurrencesToReplace = occurrencesToReplace,
occurrenceReplacer = replacer@ { occurrenceReplacer = replacer@ {
val expressionToReplace = it.elements.single() as KtExpression val expressionToReplace = it.elements.single() as KtExpression
val replacingExpression = psiFactory.createExpression(newParameterName) val replacingExpression = psiFactory.createExpression(newParameterName)
val substringInfo = expressionToReplace.extractableSubstringInfo val substringInfo = expressionToReplace.extractableSubstringInfo
@@ -213,7 +213,7 @@ public class KotlinInplaceVariableIntroducer<D extends KtCallableDeclaration> ex
public void run() { public void run() {
if (exprTypeCheckbox.isSelected()) { if (exprTypeCheckbox.isSelected()) {
String renderedType = String renderedType =
IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(myExprType); IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(myExprType);
myDeclaration.setTypeReference(new KtPsiFactory(myProject).createType(renderedType)); myDeclaration.setTypeReference(new KtPsiFactory(myProject).createType(renderedType));
} }
else { else {
@@ -86,7 +86,7 @@ class KotlinVariableInplaceIntroducer(
} }
if (expressionType != null && !noTypeInference) { if (expressionType != null && !noTypeInference) {
val renderedType = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.renderType(expressionType) val renderedType = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.renderType(expressionType)
expressionTypeCheckBox = NonFocusableCheckBox("Specify type explicitly").apply { expressionTypeCheckBox = NonFocusableCheckBox("Specify type explicitly").apply {
isSelected = false isSelected = false
setMnemonic('t') setMnemonic('t')
@@ -42,7 +42,7 @@ class KotlinMemberInfo @JvmOverloads constructor(
val isCompanionMember: Boolean = false val isCompanionMember: Boolean = false
) : MemberInfoBase<KtNamedDeclaration>(member) { ) : MemberInfoBase<KtNamedDeclaration>(member) {
companion object { companion object {
private val RENDERER = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.withOptions { private val RENDERER = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.withOptions {
modifiers = setOf(DescriptorRendererModifier.INNER) modifiers = setOf(DescriptorRendererModifier.INNER)
} }
} }
@@ -86,7 +86,7 @@ internal fun willBeUsedInSourceClass(
.all { it.element.parentsWithSelf.any { it in membersToMove } } .all { it.element.parentsWithSelf.any { it in membersToMove } }
} }
private val CALLABLE_RENDERER = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.withOptions { private val CALLABLE_RENDERER = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.withOptions {
parameterNameRenderingPolicy = ParameterNameRenderingPolicy.NONE parameterNameRenderingPolicy = ParameterNameRenderingPolicy.NONE
modifiers = emptySet() modifiers = emptySet()
startFromName = false startFromName = false
@@ -33,7 +33,7 @@ import org.jetbrains.kotlin.resolve.DescriptorToSourceUtils
fun wrapOrSkip(s: String, inCode: Boolean) = if (inCode) "<code>$s</code>" else s fun wrapOrSkip(s: String, inCode: Boolean) = if (inCode) "<code>$s</code>" else s
fun formatClassDescriptor(classDescriptor: DeclarationDescriptor) = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.render(classDescriptor) fun formatClassDescriptor(classDescriptor: DeclarationDescriptor) = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.render(classDescriptor)
fun formatPsiClass( fun formatPsiClass(
psiClass: PsiClass, psiClass: PsiClass,
@@ -102,7 +102,7 @@ class KotlinOverridingDialog extends DialogWrapper {
return KotlinBundle.message( return KotlinBundle.message(
"x.in.y", "x.in.y",
DescriptorRenderer.COMPACT.render(declarationDescriptor), DescriptorRenderer.COMPACT.render(declarationDescriptor),
IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.render(containingDescriptor) IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.render(containingDescriptor)
); );
} }
} }
@@ -29,7 +29,7 @@ import org.jetbrains.kotlin.renderer.ParameterNameRenderingPolicy
// Based on com.intellij.slicer.SliceUsageCellRenderer // Based on com.intellij.slicer.SliceUsageCellRenderer
object KotlinSliceUsageCellRenderer : SliceUsageCellRendererBase() { object KotlinSliceUsageCellRenderer : SliceUsageCellRendererBase() {
private val descriptorRenderer = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.withOptions { private val descriptorRenderer = IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.withOptions {
withDefinedIn = true withDefinedIn = true
withoutTypeParameters = true withoutTypeParameters = true
parameterNameRenderingPolicy = ParameterNameRenderingPolicy.NONE parameterNameRenderingPolicy = ParameterNameRenderingPolicy.NONE
@@ -14,7 +14,7 @@ class A {
return JetBundle.message( return JetBundle.message(
"x.in.y", "x.in.y",
DescriptorRenderer.COMPACT.render(declarationDescriptor), DescriptorRenderer.COMPACT.render(declarationDescriptor),
IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.render(containingDescriptor) IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.render(containingDescriptor)
) )
} }
} }
@@ -12,7 +12,7 @@ class A {
return JetBundle.message( return JetBundle.message(
"x.in.y", "x.in.y",
DescriptorRenderer.COMPACT.render(declarationDescriptor), DescriptorRenderer.COMPACT.render(declarationDescriptor),
IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_IN_TYPES.render(containingDescriptor) IdeDescriptorRenderers.SOURCE_CODE_SHORT_NAMES_NO_ANNOTATIONS.render(containingDescriptor)
); );
} }
} }