From 11ed783e1e1c6b8fd199bf4e79d204062032b21c Mon Sep 17 00:00:00 2001
From: "Pavel V. Talanov" <talanov.pavel@gmail.com>
Date: Thu, 29 Jan 2015 15:08:46 +0300
Subject: [PATCH] Refactor: get rid of MemberFqNameProvider

---
 .../stubBuilder/CallableClsStubBuilder.kt         |  2 +-
 .../decompiler/stubBuilder/ClassClsStubBuilder.kt |  4 ++--
 .../stubBuilder/ClsStubBuilderContext.kt          | 15 ++++-----------
 3 files changed, 7 insertions(+), 14 deletions(-)

diff --git a/idea/idea-analysis/src/org/jetbrains/kotlin/idea/decompiler/stubBuilder/CallableClsStubBuilder.kt b/idea/idea-analysis/src/org/jetbrains/kotlin/idea/decompiler/stubBuilder/CallableClsStubBuilder.kt
index 6f5f05da430..f623205a816 100644
--- a/idea/idea-analysis/src/org/jetbrains/kotlin/idea/decompiler/stubBuilder/CallableClsStubBuilder.kt
+++ b/idea/idea-analysis/src/org/jetbrains/kotlin/idea/decompiler/stubBuilder/CallableClsStubBuilder.kt
@@ -98,7 +98,7 @@ private class CallableClsStubBuilder(
     private fun doCreateCallableStub(): StubElement<out PsiElement> {
         val callableKind = Flags.CALLABLE_KIND[callableProto.getFlags()]
         val callableName = c.nameResolver.getName(callableProto.getName())
-        val callableFqName = c.memberFqNameProvider.getMemberFqName(callableName)
+        val callableFqName = c.containerFqName.child(callableName)
 
         return when (callableKind) {
             ProtoBuf.Callable.CallableKind.FUN -> {
diff --git a/idea/idea-analysis/src/org/jetbrains/kotlin/idea/decompiler/stubBuilder/ClassClsStubBuilder.kt b/idea/idea-analysis/src/org/jetbrains/kotlin/idea/decompiler/stubBuilder/ClassClsStubBuilder.kt
index b60db677e84..040a6fb7dc9 100644
--- a/idea/idea-analysis/src/org/jetbrains/kotlin/idea/decompiler/stubBuilder/ClassClsStubBuilder.kt
+++ b/idea/idea-analysis/src/org/jetbrains/kotlin/idea/decompiler/stubBuilder/ClassClsStubBuilder.kt
@@ -101,7 +101,7 @@ private class ClassClsStubBuilder(
 
     private fun doCreateClassOrObjectStub(): StubElement<out PsiElement> {
         val isClassObject = classKind == ProtoBuf.Class.Kind.CLASS_OBJECT
-        val fqName = outerContext.memberFqNameProvider.getMemberFqName(classId.getRelativeClassName().shortName())
+        val fqName = outerContext.containerFqName.child(classId.getRelativeClassName().shortName())
         val shortName = fqName.shortName()?.ref()
         val superTypeRefs = supertypeIds.filter {
             //TODO: filtering function types should go away
@@ -184,7 +184,7 @@ private class ClassClsStubBuilder(
             KotlinClassStubImpl(
                     JetStubElementTypes.ENUM_ENTRY,
                     classBody,
-                    qualifiedName = c.memberFqNameProvider.getMemberFqName(name).ref(),
+                    qualifiedName = c.containerFqName.child(name).ref(),
                     name = name.ref(),
                     superNames = array(),
                     isTrait = false,
diff --git a/idea/idea-analysis/src/org/jetbrains/kotlin/idea/decompiler/stubBuilder/ClsStubBuilderContext.kt b/idea/idea-analysis/src/org/jetbrains/kotlin/idea/decompiler/stubBuilder/ClsStubBuilderContext.kt
index c4d5de001b3..f485ee67d03 100644
--- a/idea/idea-analysis/src/org/jetbrains/kotlin/idea/decompiler/stubBuilder/ClsStubBuilderContext.kt
+++ b/idea/idea-analysis/src/org/jetbrains/kotlin/idea/decompiler/stubBuilder/ClsStubBuilderContext.kt
@@ -37,17 +37,10 @@ class ClsStubBuilderComponents(
             nameResolver: NameResolver,
             packageFqName: FqName
     ): ClsStubBuilderContext {
-        return ClsStubBuilderContext(this, nameResolver, MemberFqNameProvider(packageFqName), EmptyTypeParameters)
+        return ClsStubBuilderContext(this, nameResolver, packageFqName, EmptyTypeParameters)
     }
 }
 
-//TODO_R: remove?
-class MemberFqNameProvider(val fqName: FqName) {
-    fun getMemberFqName(name: Name): FqName = fqName.child(name)
-
-    fun child(name: Name?): MemberFqNameProvider = if (name == null) this else MemberFqNameProvider(fqName.child(name))
-}
-
 trait TypeParameters {
     fun get(id: Int): Name
 
@@ -72,7 +65,7 @@ class TypeParametersImpl(
 class ClsStubBuilderContext(
         val components: ClsStubBuilderComponents,
         val nameResolver: NameResolver,
-        val memberFqNameProvider: MemberFqNameProvider,
+        val containerFqName: FqName,
         val typeParameters: TypeParameters
 )
 
@@ -80,7 +73,7 @@ private fun ClsStubBuilderContext.child(typeParameterList: List<ProtoBuf.TypePar
     return ClsStubBuilderContext(
             this.components,
             this.nameResolver,
-            this.memberFqNameProvider.child(name),
+            if (name != null) this.containerFqName.child(name) else this.containerFqName,
             this.typeParameters.child(nameResolver, typeParameterList)
     )
 }
@@ -89,7 +82,7 @@ private fun ClsStubBuilderContext.child(nameResolver: NameResolver): ClsStubBuil
     return ClsStubBuilderContext(
             this.components,
             nameResolver,
-            this.memberFqNameProvider,
+            this.containerFqName,
             this.typeParameters
     )
 }