From: David Kastrup <dak@gnu.org>
Date: Tue, 21 Jan 2014 23:02:45 GMT
Subject: Re: [PATCH 1/2] builtin/blame.c: struct blame_entry does not need a prev link
Message-ID: <87eh41os4q.fsf@fencepost.gnu.org>
In-Reply-To: <xmqqlhy9t2z1.fsf@gitster.dls.corp.google.com>

Junio C Hamano <gitster@pobox.com> writes:

> David Kastrup <dak@gnu.org> writes:
>
>> ---
>
> Thanks.  At some point during its development I must have thought
> that having it as a dual-linked list may make it easier when we have
> to split a block into pieces, but it seems that split_overlap() does
> not need to look at this information.
>
> Needs sign-off.

Well, as I said: it's quite possible that the double-linking might be
useful for some particular hypothetical rewrite of the code.  It isn't
for the current code, and it's not useful for my own rewrite.

Will be posting a signed-off version presently.

-- 
David Kastrup