Re: [PATCH 0/4] nd/ita-empty-commit update
- From
Duy Nguyen <pclouds@gmail.com>
- Date
- Oct 25, 2016, 09:34 UTC
- Message-ID
- <CACsJy8DJqomLNB=BXSfKn4s5pmtsZffrjDJe8JspYx1MiDNsZA@mail.gmail.com>
- In-Reply-To
- <xmqqeg357hou.fsf@gitster.mtv.corp.google.com>
On Tue, Oct 25, 2016 at 12:58 AM, Junio C Hamano <gitster@pobox.com> wrote:
Show 8 quoted lines
>> The name ita-invisible-in-index is not perfect but I could not think >> of any better. Another name could be diff-cached-ignores-ita, but >> that's just half of what it does. The other half is diff-files-includes-ita... > > I can't either, and it is one of the reasons why I am reluctant. > Not being able to be named with a short-and-sweet name often is a > sign that the thing to be named is conceptually not well thought > out.
It's implementation detail leak, and probably why naming it for "normal" people is so hard. Whatever the name is must somehow imply "so these i-t-a markers actually live in the index and considered real index entries, associated to empty blob, most of the time..."
> But as we need to give it some name to the flat to ease > experimenting, let's take that name as-is.
-- Duy