Re: [WIP PATCH] diff: add option to ignore whitespaces for move detection only

From: Junio C Hamano <gitster@pobox.com>
Date: Oct 25, 2017, 01:25 UTC
Message-ID: <xmqqshe8j9we.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <20171024184809.GC79163@google.com>

Brandon Williams <bmwill@google.com> writes:

Show 5 quoted lines

> One simple idea would be to convert the single 'flag' into various bit
> fields themselves, that way if you need to add a new flag you would just
> make a new bit field.  I'm unaware of any downsides of doing so (though
> i may be missing something) but doing so would probably cause a bit of
> code churn.

The reason why people want to have their own bit in the flags word is because they want to use DIFF_OPT_{SET,CLR,TST,TOUCHED} but they do not want to do the work to extend them beyond a single word.

I think it is doable by making everything a 1-bit wide bitfield without affecting existing users.

$ 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.