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

Re: [PATCH v2 0/4] Making an elephant out of a getline() bug

From
Junio C Hamano <gitster@pobox.com>
Date
Feb 23, 2012, 19:04 UTC
Message-ID
<7vobsp4bie.fsf@alter.siamese.dyndns.org>
In-Reply-To
<cover.1329988335.git.trast@student.ethz.ch>
Thomas Rast <trast@student.ethz.ch> writes:
> So you all made very good points, and I don't want to repeat them.

I really wish you did not make an elephant out of this, or if you must, kept the minimum bugfix part that must apply to older codebase and making an elephant part clearly separable.

Anyway, I've reordered so that the bugfix part would apply to an older maintenance release, and then the remainder applied on top would merge cleanly to the bleeding edge, so there is no need to resend.

Thanks for working on this, everybody.
Previous: Thomas Rast
Message 8 of 8 in “Making an elephant out of a getline() bug”
  1. 0/4 Making an elephant out of a getline() bugThomas Rast, Feb 23, 2012
  2. 1/4 strbuf: improve strbuf_get*line documentationThomas Rast, Feb 23, 2012
  3. Jeff KingFeb 23, 2012
  4. 2/4 bundle: put strbuf_readline_fd in strbuf.c with adjustmentsThomas Rast, Feb 23, 2012
  5. 3/4 t5704: match tests to modern styleThomas Rast, Feb 23, 2012
  6. Junio C HamanoFeb 23, 2012
  7. 4/4 bundle: use a strbuf to scan the log for boundary commitsThomas Rast, Feb 23, 2012
  8. Junio C HamanoFeb 23, 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.