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

Re: [PATCH] diff: Introduce diff.algorithm variable

From
Jeff King <peff@peff.net>
Date
Mar 13, 2012, 20:48 UTC
Message-ID
<20120313204826.GA27436@sigill.intra.peff.net>
In-Reply-To
<7v399cxuk9.fsf@alter.siamese.dyndns.org>
On Tue, Mar 13, 2012 at 10:46:46AM -0700, Junio C Hamano wrote:
Show 7 quoted lines
> Tay Ray Chuan <rctay89@gmail.com> writes:
> 
> > Considering that --minimal isn't really an algorithm,...
> 
> I think this discussion thread already settled that "minimal" *is* an
> algorithm for the purpose of this topic, considering that the --minimal
> option does not apply anything but myers.

I don't know that it is settled. I proposed it as a concept, and nobody else objected. It is true that right _now_ --minimal doesn't do anything for non-myers algorithms, but in theory it could. IOW, it is a matter of the user's perspective whether the flag is "use the minimal Myers diff algorithm" versus "spend extra effort to make the diff more minimal, no matter which algorithm is being used". If the latter, it is only that we have not (yet) implemented the extra effort tweaks to the other algorithms.

So I think we could go either way.
-Peff
Previous: Junio C Hamano
Message 6 of 6 in “diff: Introduce diff.algorithm variable”
  1. diff: Introduce diff.algorithm variableMichal Privoznik, Mar 9, 2012
  2. Junio C HamanoMar 9, 2012
  3. Junio C HamanoMar 12, 2012
  4. Tay Ray ChuanMar 13, 2012
  5. Junio C HamanoMar 13, 2012
  6. Jeff KingMar 13, 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.