From b20825ad9c6263d08aefad84173bb9a2390bd833 Mon Sep 17 00:00:00 2001
From: "Dmitriy.Panov" <Dmitriy.Panov@jetbrains.com>
Date: Sat, 12 Nov 2022 03:13:40 +0100
Subject: [PATCH] IJI-967 Parameter specified as non-null is null: method
 org.jetbrains.intellij.build.impl.BackedLogger.info, parameter message

GitOrigin-RevId: f1ace38b12c09dd92dfdff70566f2a999dec9f20
---
 .../intellij/build/impl/JpsCompilationRunner.kt    | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/platform/build-scripts/src/org/jetbrains/intellij/build/impl/JpsCompilationRunner.kt b/platform/build-scripts/src/org/jetbrains/intellij/build/impl/JpsCompilationRunner.kt
index 473a2ae7dce5..1d477fbbd273 100644
--- a/platform/build-scripts/src/org/jetbrains/intellij/build/impl/JpsCompilationRunner.kt
+++ b/platform/build-scripts/src/org/jetbrains/intellij/build/impl/JpsCompilationRunner.kt
@@ -190,7 +190,7 @@ internal class JpsCompilationRunner(private val context: CompilationContext) {
       require(modules.isNotEmpty()) {
         "No modules found for artifacts ${artifacts.map { it.name }}"
       }
-      context.messages.warning("Building $modules")
+      context.messages.warning("Building ${modules.joinToString()}")
       runBuild(moduleSet = modules,
                allModules = false,
                artifactNames = artifacts.map { it.name },
@@ -426,7 +426,7 @@ private class JpsMessageHandler(private val context: CompilationContext) : Messa
   }
 }
 
-private class BackedLogger constructor(category: String?, private val fileLogger: Logger?) : DefaultLogger(category) {
+private class BackedLogger(category: String?, private val fileLogger: Logger?) : DefaultLogger(category) {
   override fun error(@Nls message: @NonNls String?, t: Throwable?, vararg details: String) {
     if (t == null) {
       messageHandler.processMessage(CompilerMessage(COMPILER_NAME, BuildMessage.Kind.ERROR, message))
@@ -442,11 +442,11 @@ private class BackedLogger constructor(category: String?, private val fileLogger
     fileLogger?.warn(message, t)
   }
 
-  override fun info(message: String) {
+  override fun info(message: String?) {
     fileLogger?.info(message)
   }
 
-  override fun info(message: String, t: Throwable?) {
+  override fun info(message: String?, t: Throwable?) {
     fileLogger?.info(message, t)
   }
 
@@ -454,7 +454,7 @@ private class BackedLogger constructor(category: String?, private val fileLogger
     return fileLogger != null && fileLogger.isDebugEnabled
   }
 
-  override fun debug(message: String) {
+  override fun debug(message: String?) {
     fileLogger?.debug(message)
   }
 
@@ -462,7 +462,7 @@ private class BackedLogger constructor(category: String?, private val fileLogger
     fileLogger?.debug(t)
   }
 
-  override fun debug(message: String, t: Throwable?) {
+  override fun debug(message: String?, t: Throwable?) {
     fileLogger?.debug(message, t)
   }
 
@@ -470,7 +470,7 @@ private class BackedLogger constructor(category: String?, private val fileLogger
     return fileLogger != null && fileLogger.isTraceEnabled
   }
 
-  override fun trace(message: String) {
+  override fun trace(message: String?) {
     fileLogger?.trace(message)
   }