# HG changeset patch # User Alexander Plavin <alexander@plav.in> # Date 1378459858 -14400 # Node ID 80633eac7b9d061d91bc0efc9fd789bc5d35946f # Parent 1935e8383a9e1bd1ac6809ad1ecafd42dd7d58b2 hgweb: eliminate extra complexity in process_dates definition There was an extra anonymous outer function, called immediately. It is removed in this patch. diff --git a/mercurial/templates/static/mercurial.js b/mercurial/templates/static/mercurial.js --- a/mercurial/templates/static/mercurial.js +++ b/mercurial/templates/static/mercurial.js @@ -176,7 +176,7 @@ } -process_dates = (function(document, RegExp, Math, isNaN, Date, _false, _true){ +function process_dates(){ // derived from code from mercurial/templatefilter.py @@ -227,9 +227,9 @@ var delta = Math.floor((now.getTime() - once.getTime()) / 1000); - var future = _false; + var future = false; if (delta < 0){ - future = _true; + future = true; delta = -delta; if (delta > (30 * scales.year)){ return "in the distant future"; @@ -253,26 +253,24 @@ } } - return function(){ - var nodes = document.getElementsByTagName('*'); - var ageclass = new RegExp('\\bage\\b'); - var dateclass = new RegExp('\\bdate\\b'); - for (var i=0; i<nodes.length; ++i){ - var node = nodes[i]; - var classes = node.className; - if (ageclass.test(classes)){ - var agevalue = age(node.textContent); - if (dateclass.test(classes)){ - // We want both: date + (age) - node.textContent += ' ('+agevalue+')'; - } else { - node.title = node.textContent; - node.textContent = agevalue; - } + var nodes = document.getElementsByTagName('*'); + var ageclass = new RegExp('\\bage\\b'); + var dateclass = new RegExp('\\bdate\\b'); + for (var i=0; i<nodes.length; ++i){ + var node = nodes[i]; + var classes = node.className; + if (ageclass.test(classes)){ + var agevalue = age(node.textContent); + if (dateclass.test(classes)){ + // We want both: date + (age) + node.textContent += ' ('+agevalue+')'; + } else { + node.title = node.textContent; + node.textContent = agevalue; } } } -})(document, RegExp, Math, isNaN, Date, false, true) +} function toggleDiffstat() { var curdetails = document.getElementById('diffstatdetails').style.display;