From 5d0243d46f3c2d369024c2641977de4cea525075 Mon Sep 17 00:00:00 2001
From: Dmitry Petrov <Dmitry.Petrov@jetbrains.com>
Date: Mon, 14 Sep 2015 14:11:23 +0300
Subject: [PATCH] Fix incremental compilation tests

---
 .../jetbrains/kotlin/codegen/PackageCodegen.java    |  1 -
 .../kotlin/codegen/state/JetTypeMapper.java         | 13 ++++++-------
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/compiler/backend/src/org/jetbrains/kotlin/codegen/PackageCodegen.java b/compiler/backend/src/org/jetbrains/kotlin/codegen/PackageCodegen.java
index d19fff65310..9aa68294bb8 100644
--- a/compiler/backend/src/org/jetbrains/kotlin/codegen/PackageCodegen.java
+++ b/compiler/backend/src/org/jetbrains/kotlin/codegen/PackageCodegen.java
@@ -40,7 +40,6 @@ import org.jetbrains.kotlin.descriptors.DeclarationDescriptor;
 import org.jetbrains.kotlin.descriptors.PackageFragmentDescriptor;
 import org.jetbrains.kotlin.diagnostics.DiagnosticUtils;
 import org.jetbrains.kotlin.fileClasses.FileClassesPackage;
-import org.jetbrains.kotlin.fileClasses.JvmFileClassInfo;
 import org.jetbrains.kotlin.load.java.JvmAbi;
 import org.jetbrains.kotlin.load.java.JvmAnnotationNames;
 import org.jetbrains.kotlin.load.kotlin.PackagePartClassUtils;
diff --git a/compiler/backend/src/org/jetbrains/kotlin/codegen/state/JetTypeMapper.java b/compiler/backend/src/org/jetbrains/kotlin/codegen/state/JetTypeMapper.java
index f3340b6371f..05e3b32695d 100644
--- a/compiler/backend/src/org/jetbrains/kotlin/codegen/state/JetTypeMapper.java
+++ b/compiler/backend/src/org/jetbrains/kotlin/codegen/state/JetTypeMapper.java
@@ -59,6 +59,7 @@ import org.jetbrains.kotlin.resolve.jvm.jvmSignature.JvmMethodParameterKind;
 import org.jetbrains.kotlin.resolve.jvm.jvmSignature.JvmMethodParameterSignature;
 import org.jetbrains.kotlin.resolve.jvm.jvmSignature.JvmMethodSignature;
 import org.jetbrains.kotlin.resolve.scopes.AbstractScopeAdapter;
+import org.jetbrains.kotlin.resolve.scopes.ChainedScope;
 import org.jetbrains.kotlin.resolve.scopes.JetScope;
 import org.jetbrains.kotlin.serialization.deserialization.DeserializedType;
 import org.jetbrains.kotlin.serialization.deserialization.descriptors.DeserializedCallableMemberDescriptor;
@@ -207,7 +208,7 @@ public class JetTypeMapper {
 
     @Nullable
     private static String getPackageMemberOwnerShortName(@NotNull DeserializedCallableMemberDescriptor descriptor) {
-        // XXX Dirty hack; see getPAckageMemberOwnerInternalName above for more details.
+        // XXX Dirty hack; see getPackageMemberOwnerInternalName above for more details.
         DeclarationDescriptor containingDeclaration = descriptor.getContainingDeclaration();
         if (containingDeclaration instanceof PackageFragmentDescriptor) {
             PackageFragmentDescriptor packageFragmentDescriptor = (PackageFragmentDescriptor) containingDeclaration;
@@ -222,15 +223,13 @@ public class JetTypeMapper {
             else if (packageFragmentDescriptor instanceof BuiltinsPackageFragment) {
                 return PackageClassUtils.getPackageClassFqName(packageFragmentDescriptor.getFqName()).shortName().asString();
             }
-            else if (scope instanceof DeserializedPackageMemberScope) {
-                // XXX IncrementalPackageScope
+            else {
+                // Incremental compilation ends up here. We do not have multifile classes support in incremental so far,
+                // so "use implementation class name" looks like a safe assumption for this case.
+                // However, this should be fixed; see getPackageMemberOwnerInternalName above for more details.
                 Name implClassName = JvmFileClassUtil.getImplClassName(descriptor);
                 return implClassName.asString();
             }
-            else {
-                throw new AssertionError("Unexpected member scope for deserialized callable: " + scope +
-                                         "; descriptor: " + descriptor);
-            }
         }
         return null;
     }