Do not mark dirty same files

Original commit: 7063e1f56a

jps/jps-plugin/src/org/jetbrains/kotlin/jps/build/FSOperationsHelper.kt

@@ -16,13 +16,18 @@
 
 package org.jetbrains.kotlin.jps.build
 
+import com.intellij.openapi.diagnostic.Logger
 import org.jetbrains.jps.ModuleChunk
 import org.jetbrains.jps.incremental.CompileContext
 import org.jetbrains.jps.incremental.FSOperations
 import org.jetbrains.jps.incremental.fs.CompilationRound
 import java.io.File
 
-class FSOperationsHelper(private val compileContext: CompileContext, private val chunk: ModuleChunk) {
+class FSOperationsHelper(
+        private val compileContext: CompileContext,
+        private val chunk: ModuleChunk,
+        private val log: Logger
+) {
     private var markedDirty = false
 
     fun hasMarkedDirty(): Boolean = markedDirty
@@ -46,11 +51,16 @@ class FSOperationsHelper(private val compileContext: CompileContext, private val
     }
 
     fun markFiles(files: Iterable<File>, excludeFiles: Set<File> = setOf()) {
-        for (file in files) {
-            if (file in excludeFiles || !file.exists()) continue
+        val filesToMark = files.toMutableSet()
+        filesToMark.removeAll(excludeFiles)
+        log.debug("Mark dirty: $filesToMark")
+
+        for (file in filesToMark) {
+            if (!file.exists()) continue
 
             FSOperations.markDirty(compileContext, CompilationRound.NEXT, file)
-            markedDirty = true
         }
+
+        markedDirty = markedDirty || filesToMark.isNotEmpty()
     }
 }

jps/jps-plugin/src/org/jetbrains/kotlin/jps/build/KotlinBuilder.kt

@@ -103,7 +103,7 @@ public class KotlinBuilder : ModuleLevelBuilder(BuilderCategory.SOURCE_PROCESSOR
     ): ModuleLevelBuilder.ExitCode {
         LOG.debug("------------------------------------------")
         val messageCollector = MessageCollectorAdapter(context)
-        val fsOperations = FSOperationsHelper(context, chunk)
+        val fsOperations = FSOperationsHelper(context, chunk, LOG)
 
         try {
             val exitCode = doBuild(chunk, context, dirtyFilesHolder, messageCollector, outputConsumer, fsOperations)
@@ -275,9 +275,8 @@ public class KotlinBuilder : ModuleLevelBuilder(BuilderCategory.SOURCE_PROCESSOR
             }
 
             if (inlineChanged) {
-                for (cache in caches) {
-                    fsOperations.markFiles(cache.getFilesToReinline(), excludeFiles = compiledFiles)
-                }
+                val files = caches.flatMap { it.getFilesToReinline() }
+                fsOperations.markFiles(files, excludeFiles = compiledFiles)
             }
         }