Replaced filterIsInstance with class-literal argument with simplified version
This commit is contained in:
@@ -42,7 +42,7 @@ public object CodegenUtilKt {
|
|||||||
if (delegateExpressionType?.isDynamic() ?: false) return mapOf();
|
if (delegateExpressionType?.isDynamic() ?: false) return mapOf();
|
||||||
|
|
||||||
return descriptor.getDefaultType().getMemberScope().getDescriptors().stream()
|
return descriptor.getDefaultType().getMemberScope().getDescriptors().stream()
|
||||||
.filterIsInstance(javaClass<CallableMemberDescriptor>())
|
.filterIsInstance<CallableMemberDescriptor>()
|
||||||
.filter { it.getKind() == CallableMemberDescriptor.Kind.DELEGATION }
|
.filter { it.getKind() == CallableMemberDescriptor.Kind.DELEGATION }
|
||||||
.toList()
|
.toList()
|
||||||
.sortBy(MemberComparator.INSTANCE as Comparator<CallableMemberDescriptor>) // Workaround for KT-6030
|
.sortBy(MemberComparator.INSTANCE as Comparator<CallableMemberDescriptor>) // Workaround for KT-6030
|
||||||
|
|||||||
@@ -118,7 +118,7 @@ public class MultiModuleJavaAnalysisCustomTest : UsefulTestCase() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
private fun checkClass(classDescriptor: ClassDescriptor) {
|
private fun checkClass(classDescriptor: ClassDescriptor) {
|
||||||
classDescriptor.getDefaultType().getMemberScope().getDescriptors().filterIsInstance(javaClass<CallableDescriptor>()).forEach {
|
classDescriptor.getDefaultType().getMemberScope().getDescriptors().filterIsInstance<CallableDescriptor>().forEach {
|
||||||
checkCallable(it)
|
checkCallable(it)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
+2
-2
@@ -23,12 +23,12 @@ import org.jetbrains.jet.lang.resolve.name.Name
|
|||||||
public class InnerClassesScopeWrapper(override val workerScope: JetScope) : AbstractScopeAdapter() {
|
public class InnerClassesScopeWrapper(override val workerScope: JetScope) : AbstractScopeAdapter() {
|
||||||
override fun getClassifier(name: Name) = workerScope.getClassifier(name) as? ClassDescriptor
|
override fun getClassifier(name: Name) = workerScope.getClassifier(name) as? ClassDescriptor
|
||||||
|
|
||||||
override fun getDeclarationsByLabel(labelName: Name) = workerScope.getDeclarationsByLabel(labelName).filterIsInstance(javaClass<ClassDescriptor>())
|
override fun getDeclarationsByLabel(labelName: Name) = workerScope.getDeclarationsByLabel(labelName).filterIsInstance<ClassDescriptor>()
|
||||||
|
|
||||||
override fun getDescriptors(kindFilter: DescriptorKindFilter,
|
override fun getDescriptors(kindFilter: DescriptorKindFilter,
|
||||||
nameFilter: (Name) -> Boolean): List<ClassDescriptor> {
|
nameFilter: (Name) -> Boolean): List<ClassDescriptor> {
|
||||||
val restrictedFilter = kindFilter.restrictedToKinds(DescriptorKindFilter.CLASSIFIERS_MASK) ?: return listOf()
|
val restrictedFilter = kindFilter.restrictedToKinds(DescriptorKindFilter.CLASSIFIERS_MASK) ?: return listOf()
|
||||||
return workerScope.getDescriptors(restrictedFilter, nameFilter).filterIsInstance(javaClass<ClassDescriptor>())
|
return workerScope.getDescriptors(restrictedFilter, nameFilter).filterIsInstance<ClassDescriptor>()
|
||||||
}
|
}
|
||||||
|
|
||||||
override fun getImplicitReceiversHierarchy(): List<ReceiverParameterDescriptor> = listOf()
|
override fun getImplicitReceiversHierarchy(): List<ReceiverParameterDescriptor> = listOf()
|
||||||
|
|||||||
+2
-2
@@ -90,7 +90,7 @@ private fun collectAllModuleInfosFromIdeaModel(project: Project): List<IdeaModul
|
|||||||
|
|
||||||
//TODO: (module refactoring) include libraries that are not among dependencies of any module
|
//TODO: (module refactoring) include libraries that are not among dependencies of any module
|
||||||
val ideaLibraries = ideaModules.flatMap {
|
val ideaLibraries = ideaModules.flatMap {
|
||||||
ModuleRootManager.getInstance(it).getOrderEntries().filterIsInstance(javaClass<LibraryOrderEntry>()).map {
|
ModuleRootManager.getInstance(it).getOrderEntries().filterIsInstance<LibraryOrderEntry>().map {
|
||||||
it.getLibrary()
|
it.getLibrary()
|
||||||
}
|
}
|
||||||
}.filterNotNull().toSet()
|
}.filterNotNull().toSet()
|
||||||
@@ -98,7 +98,7 @@ private fun collectAllModuleInfosFromIdeaModel(project: Project): List<IdeaModul
|
|||||||
val librariesInfos = ideaLibraries.map { LibraryInfo(project, it) }
|
val librariesInfos = ideaLibraries.map { LibraryInfo(project, it) }
|
||||||
|
|
||||||
val ideaSdks = ideaModules.flatMap {
|
val ideaSdks = ideaModules.flatMap {
|
||||||
ModuleRootManager.getInstance(it).getOrderEntries().filterIsInstance(javaClass<JdkOrderEntry>()).map {
|
ModuleRootManager.getInstance(it).getOrderEntries().filterIsInstance<JdkOrderEntry>().map {
|
||||||
it.getJdk()
|
it.getJdk()
|
||||||
}
|
}
|
||||||
}.filterNotNull().toSet()
|
}.filterNotNull().toSet()
|
||||||
|
|||||||
@@ -156,7 +156,7 @@ public class KotlinIndicesHelper(
|
|||||||
else {
|
else {
|
||||||
val importDirective = JetPsiFactory(project).createImportDirective(fqnString)
|
val importDirective = JetPsiFactory(project).createImportDirective(fqnString)
|
||||||
analyzeImportReference(importDirective, resolutionScope, BindingTraceContext(), module)
|
analyzeImportReference(importDirective, resolutionScope, BindingTraceContext(), module)
|
||||||
.filterIsInstance(javaClass<CallableDescriptor>())
|
.filterIsInstance<CallableDescriptor>()
|
||||||
.filter { it.getExtensionReceiverParameter() != null }
|
.filter { it.getExtensionReceiverParameter() != null }
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -189,7 +189,7 @@ public class KotlinIndicesHelper(
|
|||||||
private fun findTopLevelCallables(fqName: FqName, context: JetExpression, jetScope: JetScope): Collection<CallableDescriptor> {
|
private fun findTopLevelCallables(fqName: FqName, context: JetExpression, jetScope: JetScope): Collection<CallableDescriptor> {
|
||||||
val importDirective = JetPsiFactory(context.getProject()).createImportDirective(ImportPath(fqName, false))
|
val importDirective = JetPsiFactory(context.getProject()).createImportDirective(ImportPath(fqName, false))
|
||||||
val allDescriptors = analyzeImportReference(importDirective, jetScope, BindingTraceContext(), moduleDescriptor)
|
val allDescriptors = analyzeImportReference(importDirective, jetScope, BindingTraceContext(), moduleDescriptor)
|
||||||
return allDescriptors.filterIsInstance(javaClass<CallableDescriptor>()).filter { it.getExtensionReceiverParameter() == null }
|
return allDescriptors.filterIsInstance<CallableDescriptor>().filter { it.getExtensionReceiverParameter() == null }
|
||||||
}
|
}
|
||||||
|
|
||||||
private fun analyzeImportReference(
|
private fun analyzeImportReference(
|
||||||
|
|||||||
@@ -235,7 +235,7 @@ class SmartCompletion(val expression: JetSimpleNameExpression,
|
|||||||
if (classDescriptor != null && DescriptorUtils.isEnumClass(classDescriptor)) {
|
if (classDescriptor != null && DescriptorUtils.isEnumClass(classDescriptor)) {
|
||||||
val conditions = whenExpression.getEntries()
|
val conditions = whenExpression.getEntries()
|
||||||
.flatMap { it.getConditions().toList() }
|
.flatMap { it.getConditions().toList() }
|
||||||
.filterIsInstance(javaClass<JetWhenConditionWithExpression>())
|
.filterIsInstance<JetWhenConditionWithExpression>()
|
||||||
for (condition in conditions) {
|
for (condition in conditions) {
|
||||||
val selectorExpr = (condition.getExpression() as? JetDotQualifiedExpression)
|
val selectorExpr = (condition.getExpression() as? JetDotQualifiedExpression)
|
||||||
?.getSelectorExpression() as? JetReferenceExpression ?: continue
|
?.getSelectorExpression() as? JetReferenceExpression ?: continue
|
||||||
|
|||||||
@@ -257,7 +257,7 @@ class TypeInstantiationItems(
|
|||||||
else -> return
|
else -> return
|
||||||
}
|
}
|
||||||
val samConstructor = scope.getFunctions(`class`.getName())
|
val samConstructor = scope.getFunctions(`class`.getName())
|
||||||
.filterIsInstance(javaClass<SamConstructorDescriptor>())
|
.filterIsInstance<SamConstructorDescriptor>()
|
||||||
.singleOrNull() ?: return
|
.singleOrNull() ?: return
|
||||||
val lookupElement = LookupElementFactory.DEFAULT.createLookupElement(samConstructor, resolutionFacade, bindingContext)
|
val lookupElement = LookupElementFactory.DEFAULT.createLookupElement(samConstructor, resolutionFacade, bindingContext)
|
||||||
.assignSmartCompletionPriority(SmartCompletionItemPriority.INSTANTIATION)
|
.assignSmartCompletionPriority(SmartCompletionItemPriority.INSTANTIATION)
|
||||||
|
|||||||
+1
-1
@@ -82,7 +82,7 @@ fun findPackagePartInternalNameForLibraryFile(elementAt: JetElement): String? {
|
|||||||
}
|
}
|
||||||
|
|
||||||
val deserializedDescriptor = descriptors
|
val deserializedDescriptor = descriptors
|
||||||
.filterIsInstance(javaClass<DeserializedCallableMemberDescriptor>())
|
.filterIsInstance<DeserializedCallableMemberDescriptor>()
|
||||||
.filter { render(it) == descFromSourceText }
|
.filter { render(it) == descFromSourceText }
|
||||||
.singleOrNull()
|
.singleOrNull()
|
||||||
|
|
||||||
|
|||||||
@@ -106,7 +106,7 @@ fun navigateToPropertyOverriddenDeclarations(e: MouseEvent?, property: JetProper
|
|||||||
val renderer = DefaultPsiElementCellRenderer()
|
val renderer = DefaultPsiElementCellRenderer()
|
||||||
val navigatingOverrides = elementProcessor.getResults()
|
val navigatingOverrides = elementProcessor.getResults()
|
||||||
.sortBy(renderer.getComparator())
|
.sortBy(renderer.getComparator())
|
||||||
.filterIsInstance(javaClass<NavigatablePsiElement>())
|
.filterIsInstance<NavigatablePsiElement>()
|
||||||
|
|
||||||
PsiElementListNavigator.openTargets(e,
|
PsiElementListNavigator.openTargets(e,
|
||||||
navigatingOverrides.copyToArray(),
|
navigatingOverrides.copyToArray(),
|
||||||
|
|||||||
+2
-2
@@ -290,11 +290,11 @@ data class ControlFlow(
|
|||||||
) {
|
) {
|
||||||
val outputValueBoxer = boxerFactory(outputValues)
|
val outputValueBoxer = boxerFactory(outputValues)
|
||||||
|
|
||||||
val defaultOutputValue: ExpressionValue? = with(outputValues.filterIsInstance(javaClass<ExpressionValue>())) {
|
val defaultOutputValue: ExpressionValue? = with(outputValues.filterIsInstance<ExpressionValue>()) {
|
||||||
if (size > 1) throw IllegalArgumentException("Multiple expression values: ${outputValues.joinToString()}") else firstOrNull()
|
if (size > 1) throw IllegalArgumentException("Multiple expression values: ${outputValues.joinToString()}") else firstOrNull()
|
||||||
}
|
}
|
||||||
|
|
||||||
val jumpOutputValue: Jump? = with(outputValues.filterIsInstance(javaClass<Jump>())) {
|
val jumpOutputValue: Jump? = with(outputValues.filterIsInstance<Jump>()) {
|
||||||
when {
|
when {
|
||||||
isEmpty() ->
|
isEmpty() ->
|
||||||
null
|
null
|
||||||
|
|||||||
@@ -88,7 +88,7 @@ data class ExtractionData(
|
|||||||
it is JetDeclarationWithBody || it is JetClassInitializer
|
it is JetDeclarationWithBody || it is JetClassInitializer
|
||||||
} ?: false
|
} ?: false
|
||||||
|
|
||||||
fun getExpressions(): List<JetExpression> = originalElements.filterIsInstance(javaClass<JetExpression>())
|
fun getExpressions(): List<JetExpression> = originalElements.filterIsInstance<JetExpression>()
|
||||||
|
|
||||||
fun getCodeFragmentTextRange(): TextRange? {
|
fun getCodeFragmentTextRange(): TextRange? {
|
||||||
val originalElements = originalElements
|
val originalElements = originalElements
|
||||||
|
|||||||
+1
-1
@@ -85,7 +85,7 @@ private fun JetType.isDefault(): Boolean = KotlinBuiltIns.getInstance().isUnit(t
|
|||||||
|
|
||||||
private fun List<Instruction>.getModifiedVarDescriptors(bindingContext: BindingContext): Map<VariableDescriptor, List<JetExpression>> {
|
private fun List<Instruction>.getModifiedVarDescriptors(bindingContext: BindingContext): Map<VariableDescriptor, List<JetExpression>> {
|
||||||
val result = HashMap<VariableDescriptor, MutableList<JetExpression>>()
|
val result = HashMap<VariableDescriptor, MutableList<JetExpression>>()
|
||||||
for (instruction in filterIsInstance(javaClass<WriteValueInstruction>())) {
|
for (instruction in filterIsInstance<WriteValueInstruction>()) {
|
||||||
val expression = instruction.element as? JetExpression
|
val expression = instruction.element as? JetExpression
|
||||||
val descriptor = PseudocodeUtil.extractVariableDescriptorIfAny(instruction, false, bindingContext)
|
val descriptor = PseudocodeUtil.extractVariableDescriptorIfAny(instruction, false, bindingContext)
|
||||||
if (expression != null && descriptor != null) {
|
if (expression != null && descriptor != null) {
|
||||||
|
|||||||
@@ -145,7 +145,7 @@ public fun PsiElement.getExtractionContainers(strict: Boolean = true, includeAll
|
|||||||
return when (parent) {
|
return when (parent) {
|
||||||
is JetFile -> Collections.singletonList(parent)
|
is JetFile -> Collections.singletonList(parent)
|
||||||
is JetClassBody -> {
|
is JetClassBody -> {
|
||||||
getAllExtractionContainers(strict).filterIsInstance(javaClass<JetClassBody>())
|
getAllExtractionContainers(strict).filterIsInstance<JetClassBody>()
|
||||||
}
|
}
|
||||||
else -> {
|
else -> {
|
||||||
val enclosingDeclaration =
|
val enclosingDeclaration =
|
||||||
|
|||||||
+1
-1
@@ -75,7 +75,7 @@ public class MoveKotlinFileHandler : MoveFileHandler() {
|
|||||||
val declarationMoveProcessor = MoveKotlinTopLevelDeclarationsProcessor(
|
val declarationMoveProcessor = MoveKotlinTopLevelDeclarationsProcessor(
|
||||||
project,
|
project,
|
||||||
MoveKotlinTopLevelDeclarationsOptions(
|
MoveKotlinTopLevelDeclarationsOptions(
|
||||||
elementsToMove = psiFile.getDeclarations().filterIsInstance(javaClass<JetNamedDeclaration>()),
|
elementsToMove = psiFile.getDeclarations().filterIsInstance<JetNamedDeclaration>(),
|
||||||
moveTarget = DeferredJetFileKotlinMoveTarget(project, packageNameInfo.newPackageName) {
|
moveTarget = DeferredJetFileKotlinMoveTarget(project, packageNameInfo.newPackageName) {
|
||||||
MoveFilesOrDirectoriesUtil.doMoveFile(psiFile, newParent)
|
MoveFilesOrDirectoriesUtil.doMoveFile(psiFile, newParent)
|
||||||
newParent.findFile(psiFile.getName()) as? JetFile
|
newParent.findFile(psiFile.getName()) as? JetFile
|
||||||
|
|||||||
@@ -121,7 +121,7 @@ public abstract class AbstractKotlinSteppingTest : KotlinDebuggerTestBase() {
|
|||||||
|
|
||||||
val stepTargets = KotlinSmartStepIntoHandler().findSmartStepTargets(position)
|
val stepTargets = KotlinSmartStepIntoHandler().findSmartStepTargets(position)
|
||||||
|
|
||||||
stepTargets.filterIsInstance(javaClass<MethodSmartStepTarget>()).map {
|
stepTargets.filterIsInstance<MethodSmartStepTarget>().map {
|
||||||
stepTarget ->
|
stepTarget ->
|
||||||
when (stepTarget) {
|
when (stepTarget) {
|
||||||
is KotlinMethodSmartStepTarget -> KotlinBasicStepMethodFilter(stepTarget as KotlinMethodSmartStepTarget)
|
is KotlinMethodSmartStepTarget -> KotlinBasicStepMethodFilter(stepTarget as KotlinMethodSmartStepTarget)
|
||||||
|
|||||||
@@ -127,7 +127,7 @@ class ClassBodyConverter(private val psiClass: PsiClass,
|
|||||||
|
|
||||||
val members = convertedMembers.keySet().filter { !it.isConstructor() }
|
val members = convertedMembers.keySet().filter { !it.isConstructor() }
|
||||||
val classObjectMembers = members.filter { it !is PsiClass && it.hasModifierProperty(PsiModifier.STATIC) }
|
val classObjectMembers = members.filter { it !is PsiClass && it.hasModifierProperty(PsiModifier.STATIC) }
|
||||||
val nestedClasses = members.filterIsInstance(javaClass<PsiClass>()).filter { it.hasModifierProperty(PsiModifier.STATIC) }
|
val nestedClasses = members.filterIsInstance<PsiClass>().filter { it.hasModifierProperty(PsiModifier.STATIC) }
|
||||||
if (classObjectMembers.all { it is PsiMethod && it.hasModifierProperty(PsiModifier.PRIVATE) }) {
|
if (classObjectMembers.all { it is PsiMethod && it.hasModifierProperty(PsiModifier.PRIVATE) }) {
|
||||||
return nestedClasses.any { nestedClass -> classObjectMembers.any { converter.referenceSearcher.findMethodCalls(it as PsiMethod, nestedClass).isNotEmpty() } }
|
return nestedClasses.any { nestedClass -> classObjectMembers.any { converter.referenceSearcher.findMethodCalls(it as PsiMethod, nestedClass).isNotEmpty() } }
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -344,7 +344,7 @@ class ConstructorConverter(private val psiClass: PsiClass,
|
|||||||
private fun generateArtificialPrimaryConstructor(classBody: ClassBody): ClassBody {
|
private fun generateArtificialPrimaryConstructor(classBody: ClassBody): ClassBody {
|
||||||
assert(classBody.primaryConstructorSignature == null)
|
assert(classBody.primaryConstructorSignature == null)
|
||||||
|
|
||||||
val propertiesToInitialize = classBody.members.filterIsInstance(javaClass<Property>()).filter { it.isVal }
|
val propertiesToInitialize = classBody.members.filterIsInstance<Property>().filter { it.isVal }
|
||||||
for (function in classBody.factoryFunctions) {
|
for (function in classBody.factoryFunctions) {
|
||||||
function.body!!.updateGenerator { (codeConverter, block) ->
|
function.body!!.updateGenerator { (codeConverter, block) ->
|
||||||
// 2 cases: secondary constructor either calls another constructor or does not call any
|
// 2 cases: secondary constructor either calls another constructor or does not call any
|
||||||
|
|||||||
@@ -186,7 +186,7 @@ class Converter private(private val elementToConvert: PsiElement,
|
|||||||
private fun convertAnnotationType(psiClass: PsiClass): Class {
|
private fun convertAnnotationType(psiClass: PsiClass): Class {
|
||||||
val paramModifiers = Modifiers(listOf(Modifier.PUBLIC)).assignNoPrototype()
|
val paramModifiers = Modifiers(listOf(Modifier.PUBLIC)).assignNoPrototype()
|
||||||
val noBlankLinesInheritance = CommentsAndSpacesInheritance(blankLinesBefore = false)
|
val noBlankLinesInheritance = CommentsAndSpacesInheritance(blankLinesBefore = false)
|
||||||
val annotationMethods = psiClass.getMethods().filterIsInstance(javaClass<PsiAnnotationMethod>())
|
val annotationMethods = psiClass.getMethods().filterIsInstance<PsiAnnotationMethod>()
|
||||||
val parameters = annotationMethods
|
val parameters = annotationMethods
|
||||||
.map { method ->
|
.map { method ->
|
||||||
val returnType = method.getReturnType()
|
val returnType = method.getReturnType()
|
||||||
|
|||||||
@@ -43,7 +43,7 @@ public object EmptyReferenceSearcher: ReferenceSearcher {
|
|||||||
|
|
||||||
public object IdeaReferenceSearcher : ReferenceSearcher {
|
public object IdeaReferenceSearcher : ReferenceSearcher {
|
||||||
override fun findVariableUsages(variable: PsiVariable, scope: PsiElement): Collection<PsiReferenceExpression> {
|
override fun findVariableUsages(variable: PsiVariable, scope: PsiElement): Collection<PsiReferenceExpression> {
|
||||||
return ReferencesSearch.search(variable, LocalSearchScope(scope)).findAll().filterIsInstance(javaClass<PsiReferenceExpression>())
|
return ReferencesSearch.search(variable, LocalSearchScope(scope)).findAll().filterIsInstance<PsiReferenceExpression>()
|
||||||
}
|
}
|
||||||
|
|
||||||
override fun findMethodCalls(method: PsiMethod, scope: PsiElement): Collection<PsiMethodCallExpression> {
|
override fun findMethodCalls(method: PsiMethod, scope: PsiElement): Collection<PsiMethodCallExpression> {
|
||||||
|
|||||||
+1
-1
@@ -47,7 +47,7 @@ public class DelegationTranslator(
|
|||||||
BindingUtils.getClassDescriptor(context.bindingContext(), classDeclaration);
|
BindingUtils.getClassDescriptor(context.bindingContext(), classDeclaration);
|
||||||
|
|
||||||
private val delegationBySpecifiers =
|
private val delegationBySpecifiers =
|
||||||
classDeclaration.getDelegationSpecifiers().filterIsInstance(javaClass<JetDelegatorByExpressionSpecifier>());
|
classDeclaration.getDelegationSpecifiers().filterIsInstance<JetDelegatorByExpressionSpecifier>();
|
||||||
|
|
||||||
private class Field (val name: String, val generateField: Boolean)
|
private class Field (val name: String, val generateField: Boolean)
|
||||||
private val fields = HashMap<JetDelegatorByExpressionSpecifier, Field>();
|
private val fields = HashMap<JetDelegatorByExpressionSpecifier, Field>();
|
||||||
|
|||||||
Reference in New Issue
Block a user