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

Re: [PATCH] diff: do not reuse worktree files that need "clean" conversion

From
Junio C Hamano <gitster@pobox.com>
Date
Jul 22, 2016, 19:30 UTC
Message-ID
<xmqqinvxa2sk.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<20160722181024.GA16595@sigill.intra.peff.net>
Jeff King <peff@peff.net> writes:
> So I think we actually _can_ think of would_convert_to_git() as
> basically free. Or as free as other efficient-lookup functions we call
> like cache_name_pos().
OK.
Show 8 quoted lines
> The thought "does this tree reuse even speed things up enough to justify
> its complexity" definitely crossed my mind. It's basically swapping
> open/mmap for zlib inflating the content.
>
> But I do think it helps in the "want_file" case (i.e., when we are
> writing out a tempfile for an external command via prepare_temp_file()).
> There it helps us omit writing a tempfile to disk entirely, including
> any possible conversion.
OK.
Previous: Jeff King
Message 6 of 6 in “convert: add test to demonstrate clean invocations”
  1. convert: add test to demonstrate clean invocationslarsxschneider@gmail.com, Jul 21, 2016
  2. Jeff KingJul 21, 2016
  3. diff: do not reuse worktree files that need "clean" conversionJeff King, Jul 22, 2016
  4. Junio C HamanoJul 22, 2016
  5. Jeff KingJul 22, 2016
  6. Junio C HamanoJul 22, 2016

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.