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

Re: [PATCH, take 1] Linear-time/space rename logic (exact renames only)

From
Jeff King <peff@peff.net>
Date
Oct 22, 2007, 07:21 UTC
Message-ID
<20071022072153.GA6205@coredump.intra.peff.net>
In-Reply-To
<20071022070750.GM1179MdfPADPa@greensroom.kotnet.org>
On Mon, Oct 22, 2007 at 09:07:50AM +0200, Sven Verdoolaege wrote:
> Aren't you truncating the ptr list after the first entry here?
> (While you still need the whole list in free_file_table.)

Yes, good eyes. And because we actually reverse the list, it's not as simple as just sticking the two broken up pieces together again; the original head must end up as the head of the list after they are glued together again, but it is actually the tail of one of the lists.

-Peff
Previous: Sven VerdoolaegeNext: Linus Torvalds
Message 6 of 13 in “Linear-time/space rename logic (exact renames only)”
  1. Linear-time/space rename logic (exact renames only)Linus Torvalds, Oct 21, 2007
  2. David SymondsOct 22, 2007
  3. Linus TorvaldsOct 22, 2007
  4. Jeff KingOct 22, 2007
  5. Sven VerdoolaegeOct 22, 2007
  6. Jeff KingOct 22, 2007
  7. Linus TorvaldsOct 22, 2007
  8. Linear-time/space rename logic (exact renames only)Linus Torvalds, Oct 22, 2007
  9. Linus TorvaldsOct 22, 2007
  10. Linus TorvaldsOct 22, 2007
  11. Alex RiesenOct 22, 2007
  12. Linus TorvaldsOct 22, 2007
  13. Alex RiesenOct 22, 2007

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.