From fe38069e39160a1a0342de9152cfb977b93cf620 Mon Sep 17 00:00:00 2001 From: Pieter de Bie Date: Thu, 9 Oct 2008 21:32:18 +0200 Subject: [PATCH] HTML: Move tab replacement to start of diffhighlight This might improve performance, as we don't have to match again every line. --- html/diffHighlighter.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/html/diffHighlighter.js b/html/diffHighlighter.js index fc0f1e4..c74bea4 100644 --- a/html/diffHighlighter.js +++ b/html/diffHighlighter.js @@ -14,7 +14,7 @@ var highlightDiffs = function() { for (var diffn = 0; diffn < diffs.length; diffn++) { var diff = diffs[diffn]; - var content = diff.innerHTML.replace(/\s+$/, ''); + var content = diff.innerHTML.replace(/\t/g, " ");; var line1 = ""; var line2 = ""; @@ -42,7 +42,7 @@ var highlightDiffs = function() { continue; } - l = l.replace(/\t/g, " "); + if (firstChar == "+") { // Highlight trailing whitespace if (m = l.match(/\s+$/))