Patchwork [5,of,7,V3] hgweb: add a 'linerange' parameter to webutil.diffs()

mail settings
Submitter Denis Laxalde
Date March 22, 2017, 10:37 a.m.
Message ID <>
Download mbox | patch
Permalink /patch/19571/
State Changes Requested
Headers show


Denis Laxalde - March 22, 2017, 10:37 a.m.
# HG changeset patch
# User Denis Laxalde <>
# Date 1489414549 -3600
#      Mon Mar 13 15:15:49 2017 +0100
# Node ID 18d413236b1670c476584c4a6d6e103639ae7996
# Parent  1fb2e7f3ed38dc79379cccb04133e745fd5825fa
# Available At
#              hg pull -r 18d413236b16
# EXP-Topic linerange-log/hgweb-filelog
hgweb: add a 'linerange' parameter to webutil.diffs()

This is used to filter out hunks based on their range (with respect to 'node2'
for patch.diffhunks() call, i.e. 'ctx' for webutil.diffs()).

This is the simplest way to filter diff hunks, here done on server side. Later
on, it might be interesting to perform this filtering on client side and
expose a "toggle" action to alternate between full and filtered diff.


diff --git a/mercurial/hgweb/ b/mercurial/hgweb/
--- a/mercurial/hgweb/
+++ b/mercurial/hgweb/
@@ -434,7 +434,7 @@  def listfilediffs(tmpl, files, node, max
     if len(files) > max:
         yield tmpl('fileellipses')
-def diffs(repo, tmpl, ctx, basectx, files, style):
+def diffs(repo, tmpl, ctx, basectx, files, style, linerange=None):
     def prettyprintlines(lines, blockno):
         for lineno, l in enumerate(lines, 1):
@@ -468,6 +468,11 @@  def diffs(repo, tmpl, ctx, basectx, file
             header = header[1:]
         lines = [h + '\n' for h in header]
         for hunkrange, hunklines in hunks:
+            if linerange is not None and hunkrange is not None:
+                s1, l1, s2, l2 = hunkrange
+                lb, ub = linerange
+                if not (lb <= s2 < ub or lb < s2 + l2 <= ub):
+                    continue
         if lines:
             yield tmpl('diffblock', blockno=blockno,