From: Felipe Contreras <felipe.contreras@gmail.com>
Date: Tue, 03 Sep 2013 21:52:19 GMT
Subject: Re: [PATCH 3/4] t: rev-parse-parents: avoid yoda conditions
Message-ID: <CAMP44s0zRZDGwd6KcVi81JEDQv38THCJctA8zUZiYht-uqY4vA@mail.gmail.com>
In-Reply-To: <xmqq8uzdu817.fsf@gitster.dls.corp.google.com>

On Tue, Sep 3, 2013 at 12:31 PM, Junio C Hamano <gitster@pobox.com> wrote:
> Felipe Contreras <felipe.contreras@gmail.com> writes:
>
>> There are two ways to fix an inconsistency, the other way is to fix
>> test_cmp. But that would be a change, and change is not welcome in
>> Git.
>
> If you want to do "test_cmp $actual $expect", you would have to
> first "fix" people's expectation that "diff A B" produces a change
> necessary to bring A to B, which would not likely to happen.  We do
> the 'test_cmp expect actual' for a reason.

No, you just do "diff B A".

-- 
Felipe Contreras