From: Junio C Hamano <gitster@pobox.com>
Date: Fri, 08 Mar 2013 21:46:46 GMT
Subject: Re: [PATCH v2 3/4] status: show more info than "currently not on any branch"
Message-ID: <7vtxolfttl.fsf@alter.siamese.dyndns.org>
In-Reply-To: <CACsJy8CnDoMRzR5ifH3+0AKRLfzrNpkc3-NOxoA85SiRUb40ng@mail.gmail.com>

Duy Nguyen <pclouds@gmail.com> writes:

>> It should be more like this, I would think:
>>
>>         for_each_recent_reflog_ent();
>>         if (!found)
>>                 for_each_reflog_ent();
>>         if (!found)
>>                 return;
>
> Yes. This "recent" optimization is tricky.

Not really.  What is tricky is that reflog is an append-only file
and we only have an API to let us read it in the oldest to newer
order, which is natural for the file format, but unsuited for the
purpose of finding out nth most recent anything.

See the other thread I am going to send out soon on this.