From 6ff2d682064262d52c4fa5558c8c9314d7d93ef1 Mon Sep 17 00:00:00 2001 From: Andrey Breslav Date: Thu, 7 Nov 2013 16:09:17 +0400 Subject: [PATCH] A flag replaced by a predicate --- .../resolve/java/lazy/descriptors/LazyJavaClassMemberScope.kt | 2 +- .../java/lazy/descriptors/LazyJavaPackageFragmentScope.kt | 2 +- .../jet/lang/resolve/java/lazy/descriptors/MemberIndex.kt | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/core/descriptor.loader.java/src/org/jetbrains/jet/lang/resolve/java/lazy/descriptors/LazyJavaClassMemberScope.kt b/core/descriptor.loader.java/src/org/jetbrains/jet/lang/resolve/java/lazy/descriptors/LazyJavaClassMemberScope.kt index 955b05e9904..ad090406cca 100644 --- a/core/descriptor.loader.java/src/org/jetbrains/jet/lang/resolve/java/lazy/descriptors/LazyJavaClassMemberScope.kt +++ b/core/descriptor.loader.java/src/org/jetbrains/jet/lang/resolve/java/lazy/descriptors/LazyJavaClassMemberScope.kt @@ -57,7 +57,7 @@ public class LazyJavaClassMemberScope( private val jClass: JavaClass ) : LazyJavaMemberScope(c, containingDeclaration) { - override fun computeMemberIndex(): MemberIndex = object : ClassMemberIndex(jClass, mustBeStatic = false) { + override fun computeMemberIndex(): MemberIndex = object : ClassMemberIndex(jClass, { m -> !m.isStatic() }) { // For SAM-constructors override fun getAllMetodNames(): Collection = super.getAllMetodNames() + getAllClassNames() } diff --git a/core/descriptor.loader.java/src/org/jetbrains/jet/lang/resolve/java/lazy/descriptors/LazyJavaPackageFragmentScope.kt b/core/descriptor.loader.java/src/org/jetbrains/jet/lang/resolve/java/lazy/descriptors/LazyJavaPackageFragmentScope.kt index 4ba230737d3..56aa8faa4e2 100644 --- a/core/descriptor.loader.java/src/org/jetbrains/jet/lang/resolve/java/lazy/descriptors/LazyJavaPackageFragmentScope.kt +++ b/core/descriptor.loader.java/src/org/jetbrains/jet/lang/resolve/java/lazy/descriptors/LazyJavaPackageFragmentScope.kt @@ -87,7 +87,7 @@ public class LazyPackageFragmentScopeForJavaClass( packageFragment: LazyJavaPackageFragment ) : LazyJavaPackageFragmentScope(c, packageFragment) { - override fun computeMemberIndex(): MemberIndex = computeMemberIndexForSamConstructors(ClassMemberIndex(jClass, mustBeStatic = true)) + override fun computeMemberIndex(): MemberIndex = computeMemberIndexForSamConstructors(ClassMemberIndex(jClass, { m -> m.isStatic() })) // nested classes are loaded as members of their outer classes, not packages override fun getAllClassNames(): Collection = listOf() diff --git a/core/descriptor.loader.java/src/org/jetbrains/jet/lang/resolve/java/lazy/descriptors/MemberIndex.kt b/core/descriptor.loader.java/src/org/jetbrains/jet/lang/resolve/java/lazy/descriptors/MemberIndex.kt index 43eb5ae7ad3..f9894678678 100644 --- a/core/descriptor.loader.java/src/org/jetbrains/jet/lang/resolve/java/lazy/descriptors/MemberIndex.kt +++ b/core/descriptor.loader.java/src/org/jetbrains/jet/lang/resolve/java/lazy/descriptors/MemberIndex.kt @@ -41,10 +41,10 @@ object EMPTY_MEMBER_INDEX : MemberIndex { override fun getAllFieldNames() = listOf() } -open class ClassMemberIndex(val jClass: JavaClass, mustBeStatic: Boolean) : MemberIndex { +open class ClassMemberIndex(val jClass: JavaClass, filter: (JavaMember) -> Boolean) : MemberIndex { private val memberFilter = { (m: JavaMember) -> - m.isStatic() == mustBeStatic && + filter(m) && m.getVisibility() != Visibilities.PRIVATE }