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

Re: What's cooking in git.git (Jun 2017, #06; Thu, 22)

From
Junio C Hamano <gitster@pobox.com>
Date
Jun 24, 2017, 00:37 UTC
Message-ID
<xmqq60fmb4q8.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<CAGZ79kZyMUKPahTXOF7ds6zSvFKLrM2N78JKG-2+GRL5nnmuvA@mail.gmail.com>
Stefan Beller <sbeller@google.com> writes:
> * add some heuristic to omit small blobs, (empty lines, closing braces)
>   Maybe this is can be solved by not considering anything
>   that occurs multiple times?

I vaguely recall that we had to do something similar in "blame -C" where it tries to avoid passing blame for insignificant changes down as copied. But for the purpose of this "moved line coloring", excluding multiple copy destinations of the same thing may be a simpler and more robust solution. It will not catch "somebody stupidly removed one function and made two private copies", though.

Previous: Stefan BellerNext: Junio C Hamano
Message 8 of 12 in “What's cooking in git.git (Jun 2017, #06; Thu, 22)”
  1. Junio C HamanoJun 22, 2017
  2. Ævar Arnfjörð BjarmasonJun 22, 2017
  3. Junio C HamanoJun 23, 2017
  4. Junio C HamanoJun 24, 2017
  5. Lars SchneiderJun 23, 2017
  6. Ævar Arnfjörð BjarmasonJun 23, 2017
  7. Stefan BellerJun 23, 2017
  8. Junio C HamanoJun 24, 2017
  9. Junio C HamanoJun 24, 2017
  10. Stefan BellerJun 27, 2017
  11. Junio C HamanoJun 24, 2017
  12. Stefan BellerJun 24, 2017

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.