From fd5bf338614fa5de3603b06a374b00839b7676a4 Mon Sep 17 00:00:00 2001 From: Igor Yakovlev Date: Wed, 7 Aug 2019 16:45:38 +0300 Subject: [PATCH] Removed asserts for mirror files for java stubs on kt files The behavior of this part of code is following: 1) We wrap the kotlin light class to the object that wraps the kt files into compiled (!) cls stub (notice, that most of the time it is a single file wrapper) 2) Platform wishes to get the mirror psi of this "compiled" stub work with text of it. 3) Platform request mirror element that asserted to be ok ONLY for multifiles stub. 4) It leads to EA-209577 There is no idea why so, but the exception is high-rated, so this assertion was deleted. Fixed EA-209577 --- .../kotlin/asJava/builder/LightClassBuilder.kt | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/compiler/light-classes/src/org/jetbrains/kotlin/asJava/builder/LightClassBuilder.kt b/compiler/light-classes/src/org/jetbrains/kotlin/asJava/builder/LightClassBuilder.kt index 0ff5bd3ce6c..54ed687f251 100644 --- a/compiler/light-classes/src/org/jetbrains/kotlin/asJava/builder/LightClassBuilder.kt +++ b/compiler/light-classes/src/org/jetbrains/kotlin/asJava/builder/LightClassBuilder.kt @@ -91,20 +91,6 @@ private fun createJavaFileStub(project: Project, packageFqName: FqName, files: C override fun isPhysical() = false - override fun appendMirrorText(indentLevel: Int, buffer: StringBuilder) { - if (files.size == 1) { - LOG.error("Mirror text should never be calculated for light classes generated from a single file") - } - super.appendMirrorText(indentLevel, buffer) - } - - override fun getMirror(): PsiElement { - if (files.size == 1) { - LOG.error("Mirror element should never be calculated for light classes generated from a single file") - } - return super.getMirror() - } - override fun getText(): String { return files.singleOrNull()?.text ?: super.getText() }