Completion page; avoid building many separate strings.

1 files 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 ""