From 97e5da6cbdcf1b5cf4f80cf550fc26c1ce194a3e Mon Sep 17 00:00:00 2001
From: Stephen Blott
Date: Thu, 4 Jun 2015 05:30:13 +0100
Subject: Completion page; avoid building many separate strings.

---
 pages/completion_engines.coffee | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/pages/completion_engines.coffee b/pages/completion_engines.coffee
index 9fd5bbe0..94381fb5 100644
--- a/pages/completion_engines.coffee
+++ b/pages/completion_engines.coffee
@@ -6,27 +6,27 @@ cleanUpRegexp = (re) ->
     .replace /\\\//g, "/"
 
 DomUtils.documentReady ->
-  html = ""
+  html = []
   for engine in CompletionEngines[0...CompletionEngines.length-1]
     engine = new engine
-    html += "<h4>#{engine.constructor.name}</h4>\n"
-    html += "<div class=\"engine\">"
+    html.push "<h4>#{engine.constructor.name}</h4>\n"
+    html.push "<div class=\"engine\">"
     if engine.regexps
-      html += "<pre>"
-      html += "#{cleanUpRegexp re}\n" for re in engine.regexps
-      html += "</pre>"
+      html.push "<pre>"
+      html.push "#{cleanUpRegexp re}\n" for re in engine.regexps
+      html.push "</pre>"
     if engine.prefix
-      html += "<p>This uses the general Google completion engine, but adds the prefix \"<tt>#{engine.prefix.trim()}</tt>\" to the query.</p>"
+      html.push "<p>This uses the general Google completion engine, but adds the prefix \"<tt>#{engine.prefix.trim()}</tt>\" to the query.</p>"
     if engine.exampleSearchUrl and engine.exampleKeyword
       engine.exampleDescription ||= engine.constructor.name
-      html += "<p>"
-      html += "Example:"
-      html += "<pre>"
-      html += "#{engine.exampleKeyword}: #{engine.exampleSearchUrl} #{engine.exampleDescription}"
-      html += "</pre>"
-      html += "</p>"
-    html += "</div>"
+      html.push "<p>"
+      html.push "Example:"
+      html.push "<pre>"
+      html.push "#{engine.exampleKeyword}: #{engine.exampleSearchUrl} #{engine.exampleDescription}"
+      html.push "</pre>"
+      html.push "</p>"
+    html.push "</div>"
 
-  document.getElementById("engineList").innerHTML = html
+  document.getElementById("engineList").innerHTML = html.join ""
 
 
-- 
cgit v1.2.3