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

Re: [JGIT PATCH 8/6] Fix zero context insert and delete hunk headers to match CGit

From
Miles Bader <miles@gnu.org>
Date
May 3, 2009, 00:50 UTC
Message-ID
<8763gjdn6r.fsf@catnip.gol.com>
In-Reply-To
<20090503001423.GO23604@spearce.org>
"Shawn O. Pearce" <spearce@spearce.org> writes:
>   And here is that hack.  It just feels wrong to me that I need
>   to subtract 1 from the Edit region's line numbers, *only* when
>   context is 0, in order to get the same output as CGit.

A big comment saying "this may look a bit funny, but it's the standard's fault: [text from std...]" might help salve the wound...

-Miles
-- 
.Numeric stability is probably not all that important when you're guessing.
Previous: Shawn O. PearceNext: Robin Rosenberg
Message 3 of 8 in “BROKEN: Add a zero line context test for diff.DiffFormatter”
  1. 7/6 BROKEN: Add a zero line context test for diff.DiffFormatterShawn O. Pearce, May 3, 2009
  2. 8/6 Fix zero context insert and delete hunk headers to match CGitShawn O. Pearce, May 3, 2009
  3. Miles BaderMay 3, 2009
  4. Robin RosenbergMay 3, 2009
  5. Ferry Huberts (Pelagic)May 3, 2009
  6. Robin RosenbergMay 3, 2009
  7. Ferry Huberts (Pelagic)May 3, 2009
  8. SZEDER GáborMay 5, 2009

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.