git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v4 0/8] Highlight interesting parts of diff

From
Michał Kiedrowicz <michal.kiedrowicz@gmail.com>
Date
Apr 11, 2012, 21:18 UTC
Message-ID
<1334179124-14258-1-git-send-email-michal.kiedrowicz@gmail.com>

Reading diff output is sometimes very hard, even if it's colored, especially if lines differ only in few characters. This is often true when a commit fixes a typo or renames some variables or functions.

This patch series teaches gitweb to highlight fragments that are different between old and new line. This should mimic the same feature in Trac or GitHub.

Changes since v3:
1) gitweb: Use descriptive names in esc_html_hl_regions()
    * Renamed $beg in $begin
2) gitweb: Extract print_sidebyside_diff_lines()
    * Reordered check for !$class
    * Reworded commit message to better (I hope) explain why the conditions
      were changed
    * Dropped a comment '# assume that it is change'
3) gitweb: Use print_diff_chunk() for both side-by-side and inline diffs
    * Added ', and at the end of hunk.' to the commit message
Jakub Narębski (1):
  gitweb: Pass esc_html_hl_regions() options to esc_html()
Michał Kiedrowicz (7):
  gitweb: Use descriptive names in esc_html_hl_regions()
  gitweb: esc_html_hl_regions(): Don't create empty <span> elements
  gitweb: Extract print_sidebyside_diff_lines()
  gitweb: Use print_diff_chunk() for both side-by-side and inline diffs
  gitweb: Push formatting diff lines to print_diff_chunk()
  gitweb: Highlight interesting parts of diff
  gitweb: Refinement highlightning in combined diffs
 gitweb/gitweb.perl       |  323 +++++++++++++++++++++++++++++++++-------------
 gitweb/static/gitweb.css |    8 +
 2 files changed, 244 insertions(+), 87 deletions(-)
-- 
1.7.8.4
Next: Michał Kiedrowicz
Message 1 of 10 in “Highlight interesting parts of diff”
  1. 0/8 Highlight interesting parts of diffMichał Kiedrowicz, Apr 11, 2012
  2. 1/8 gitweb: Use descriptive names in esc_html_hl_regions()Michał Kiedrowicz, Apr 11, 2012
  3. 2/8 gitweb: esc_html_hl_regions(): Don't create empty <span> elementsMichał Kiedrowicz, Apr 11, 2012
  4. 3/8 gitweb: Pass esc_html_hl_regions() options to esc_html()Michał Kiedrowicz, Apr 11, 2012
  5. 4/8 gitweb: Extract print_sidebyside_diff_lines()Michał Kiedrowicz, Apr 11, 2012
  6. 5/8 gitweb: Use print_diff_chunk() for both side-by-side and inline diffsMichał Kiedrowicz, Apr 11, 2012
  7. 6/8 gitweb: Push formatting diff lines to print_diff_chunk()Michał Kiedrowicz, Apr 11, 2012
  8. 7/8 gitweb: Highlight interesting parts of diffMichał Kiedrowicz, Apr 11, 2012
  9. 8/8 gitweb: Refinement highlightning in combined diffsMichał Kiedrowicz, Apr 11, 2012
  10. Junio C HamanoApr 11, 2012

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.