From: Junio C Hamano <gitster@pobox.com>
Date: Fri, 07 Mar 2014 22:16:27 GMT
Subject: Re: [PATCH v7 03/11] trailer: read and process config information
Message-ID: <xmqq4n39u01w.fsf@gitster.dls.corp.google.com>
In-Reply-To: <87ob1hbrfp.fsf@fencepost.gnu.org>

David Kastrup <dak@gnu.org> writes:

> Junio C Hamano <gitster@pobox.com> writes:
>
>> Christian Couder <chriscool@tuxfamily.org> writes:
>>
>>> The config information is stored in the list
>>> whose first item is pointed to by:
>>>
>>> static struct trailer_item *first_conf_item;
>>
>> If feels somewhat strange ...
>
> Can't say I agree here.  Basically all my doubly-linked lists are not
> for traversing data forwards and backwards ...
> Having a "last" pointer is an orthogonal concept ...

Yeah, that is where "somewhat strange", not "wrong", comes from ;-)