Merge pull request #595 from eed3si9n/bport/replace-literally

[1.2.x] Avoid regex in UsedName for the common case of names without newlines
sbt · Sep 7, 2018 · 932472b · 932472b
2 parents 335878c + 6f909dc
commit 932472b
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/internal/zinc-core/src/main/scala/sbt/internal/inc/UsedName.scala b/internal/zinc-core/src/main/scala/sbt/internal/inc/UsedName.scala
@@ -23,6 +23,9 @@ object UsedName {
   }
 
   private def escapeControlChars(name: String) = {
-    name.replaceAllLiterally("\n", "\u26680A")
+    if (name.indexOf('\n') > 0) // optimize for common case to regex overhead
+      name.replaceAllLiterally("\n", "\u26680A")
+    else
+      name
   }
 }