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

Re: [PATCH] Adjust diff-helper to diff-tree -v -z changes.

From
Linus Torvalds <torvalds@osdl.org>
Date
May 29, 2005, 18:31 UTC
Message-ID
<Pine.LNX.4.58.0505291129520.10545@ppc970.osdl.org>
In-Reply-To
<7vvf54xael.fsf_-_@assigned-by-dhcp.cox.net>
On Fri, 27 May 2005, Junio C Hamano wrote:
Show 7 quoted lines
>
> The latest change to diff-tree -z output adds an extra line
> termination after non diff-raw material (the header and the
> commit message).  To compensate for this change, stop adding the
> output termination of our own.  "diff-tree -v -z" piped to
> "diff-helper -z" would give different result from "diff-tree -v"
> piped to "diff-helper" without this change.
I think this is really a bug in your "read_line()" interface.
You should include the terminating character in the line count.
This also fixes and simplifies "EOF" handling.
		Linus
Previous: Junio C HamanoNext: Junio C Hamano
Message 12 of 13 in “Fix ptrdiff_t vs. int”
  1. Fix ptrdiff_t vs. intMarkus F.X.J. Oberhumer, May 27, 2005
  2. Danjel McGouganMay 27, 2005
  3. Morten WelinderMay 27, 2005
  4. H. Peter AnvinMay 27, 2005
  5. Linus TorvaldsMay 27, 2005
  6. Linus TorvaldsMay 27, 2005
  7. Markus F.X.J. OberhumerMay 27, 2005
  8. H. Peter AnvinMay 27, 2005
  9. Linus TorvaldsMay 27, 2005
  10. Junio C HamanoMay 27, 2005
  11. Adjust diff-helper to diff-tree -v -z changes.Junio C Hamano, May 28, 2005
  12. Linus TorvaldsMay 29, 2005
  13. Junio C HamanoMay 29, 2005

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.