Merge pull request #1888 from chriswebster/fe

Switch for comprehension + `to` range method to foreach call. This was imposing somewhere between a 12-18% performance penalty.
lift · Aug 5, 2017 · db05d86 · db05d86
2 parents f7ace67 + 0c3e066
commit db05d86
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/core/json/src/main/scala/net/liftweb/json/JsonAST.scala b/core/json/src/main/scala/net/liftweb/json/JsonAST.scala
@@ -817,8 +817,7 @@ object JsonAST {
   }
 
   private def appendEscapedString(buf: Appendable, s: String, settings: RenderSettings) {
-    for (i <- 0 until s.length) {
-      val c = s.charAt(i)
+    s.foreach { c =>
       val strReplacement = c match {
         case '"'  => "\\\""
         case '\\' => "\\\\"