KT-12877: minor fixes and cleanup

This commit is contained in:
Alexey Andreev
2016-11-21 15:31:04 +03:00
committed by Alexey Andreev
parent d1820b42ef
commit d40e923722
5 changed files with 3 additions and 6 deletions
@@ -16,7 +16,6 @@
package org.jetbrains.kotlin.js.analyze package org.jetbrains.kotlin.js.analyze
import com.intellij.openapi.util.text.StringUtil
import org.jetbrains.kotlin.config.CommonConfigurationKeys import org.jetbrains.kotlin.config.CommonConfigurationKeys
import org.jetbrains.kotlin.config.LanguageVersionSettingsImpl import org.jetbrains.kotlin.config.LanguageVersionSettingsImpl
import org.jetbrains.kotlin.context.ContextForNewModule import org.jetbrains.kotlin.context.ContextForNewModule
@@ -48,8 +48,6 @@ private val DIAGNOSTIC_FACTORY_TO_RENDERER by lazy {
put(ErrorsJs.JS_NAME_PROHIBITED_FOR_OVERRIDE, "@JsName is prohibited for overridden members") put(ErrorsJs.JS_NAME_PROHIBITED_FOR_OVERRIDE, "@JsName is prohibited for overridden members")
put(ErrorsJs.JS_NAME_PROHIBITED_FOR_EXTENSION_PROPERTY, "@JsName is prohibited for extension properties") put(ErrorsJs.JS_NAME_PROHIBITED_FOR_EXTENSION_PROPERTY, "@JsName is prohibited for extension properties")
put(ErrorsJs.JS_NAME_PROHIBITED_FOR_NAMED_NATIVE, "@JsName is prohibited for @native declaration with explicit name") put(ErrorsJs.JS_NAME_PROHIBITED_FOR_NAMED_NATIVE, "@JsName is prohibited for @native declaration with explicit name")
put(ErrorsJs.JS_MODULE_PROHIBITED_ON_VAR, "@JsModule annotation prohibited for 'var' declarations. Use 'val' instead.")
put(ErrorsJs.JS_MODULE_PROHIBITED_ON_NON_NATIVE, "@JsModule annotation prohibited for non-@native declarations.")
put(ErrorsJs.JS_MODULE_PROHIBITED_ON_VAR, "@JsModule and @JsNonModule annotations prohibited for 'var' declarations. " + put(ErrorsJs.JS_MODULE_PROHIBITED_ON_VAR, "@JsModule and @JsNonModule annotations prohibited for 'var' declarations. " +
"Use 'val' instead.") "Use 'val' instead.")
put(ErrorsJs.JS_MODULE_PROHIBITED_ON_NON_NATIVE, "@JsModule and @JsNonModule annotations prohibited for non-@native declarations.") put(ErrorsJs.JS_MODULE_PROHIBITED_ON_NON_NATIVE, "@JsModule and @JsNonModule annotations prohibited for non-@native declarations.")
@@ -230,7 +230,7 @@ public final class AnnotationsUtils {
} }
@NotNull @NotNull
public static List<AnnotationDescriptor> getContainingFileAnnotations( private static List<AnnotationDescriptor> getContainingFileAnnotations(
@NotNull BindingContext bindingContext, @NotNull BindingContext bindingContext,
@NotNull DeclarationDescriptor descriptor @NotNull DeclarationDescriptor descriptor
) { ) {
@@ -50,7 +50,7 @@ class KotlinJavascriptPackageFragment(
loadResource(KotlinJavascriptSerializedResourcePaths.getFileListFilePath(fqName))?.use { rawInput -> loadResource(KotlinJavascriptSerializedResourcePaths.getFileListFilePath(fqName))?.use { rawInput ->
val input = CodedInputStream.newInstance(rawInput) val input = CodedInputStream.newInstance(rawInput)
val filesProto = JsProtoBuf.Files.parseFrom(input).fileOrBuilderList val filesProto = JsProtoBuf.Files.parseFrom(input).fileOrBuilderList
filesProto.map { it.id to FileHolder(it.annotationList) }.toMap() filesProto.associate { it.id to FileHolder(it.annotationList) }
}.orEmpty() }.orEmpty()
} }
@@ -240,7 +240,7 @@ public final class StaticContext {
@NotNull @NotNull
private JsExpression buildQualifiedExpression(@NotNull DeclarationDescriptor descriptor) { private JsExpression buildQualifiedExpression(@NotNull DeclarationDescriptor descriptor) {
if (descriptor instanceof ClassDescriptor) { if (descriptor instanceof ClassDescriptor) {
ClassDescriptor classDescriptor = (ClassDescriptor) descriptor; ClassDescriptor classDescriptor = (ClassDescriptor) descriptor;
if (KotlinBuiltIns.isAny(classDescriptor)) { if (KotlinBuiltIns.isAny(classDescriptor)) {
return pureFqn("Object", null); return pureFqn("Object", null);