Subject: Re: [PATCH v5 0/8] port the filtering part of branch.c to use ref-filter APIs

From: Matthieu Moy <[email-masked]>
Date: Mon, 21 Sep 2015 08:23:18 +0200

Karthik Nayak <[email-masked]> writes:

> On Mon, Sep 21, 2015 at 12:30 AM, Matthieu Moy
> <[email-masked]> wrote:
>> Karthik Nayak <[email-masked]> writes:
>>
>>> --- a/t/t3203-branch-output.sh
>>> +++ b/t/t3203-branch-output.sh
>>> @@ -145,8 +145,8 @@ EOF
>>>
>>>  test_expect_success 'git branch `--sort` option' '
>>>       cat >expect <<-\EOF &&
>>> -     * (HEAD detached from fromtag)
>>>         branch-two
>>> +     * (HEAD detached from fromtag)
>>>         branch-one
>>>         master
>>>       EOF
>>
>> I thought this was obviously wrong, but this tests uses
>> --sort=objectsize, so it's OK. Indeed, HEAD and branch-one have
>> objectsize=215, and branch-two has objectsize=167.
>>
>> So, the previous code was hardcoding "detached HEAD always comes first",
>> but we now sort the whole list, including detached HEAD, so branch-two
>> comes first.
>>
>
> Yeah! Even Junio said this was better, and even I think this is what
> the user would expect.

Yes, that was not an objection, just me thinking aloud during my review.

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/