Re: [PATCH 1/2] transport-helper: report errors properly

From: Felipe Contreras <felipe.contreras@gmail.com>
Date: Apr 14, 2013, 15:54 UTC
Message-ID: <CAMP44s2Ksi_nNm8f+YTdceSSavtiPODHKw-4eASeEeCWW2N91g@mail.gmail.com>
In-Reply-To: <7v38ut7ilq.fsf@alter.siamese.dyndns.org>

On Sun, Apr 14, 2013 at 12:23 AM, Junio C Hamano <gitster@pobox.com> wrote:

> "Double" may only be showing that we do not have enough trusted
> maintainers; ideally I would like it to have "Triple" or more.

A double or triple review raises *a single* standard higher, but having more than one standard is never good. But apparently your trust in Jeff means more than caring about the double standard.

But good to know, my patches will always have an unfair disadvantages to everybody else's.

>  - when we add something, we try not to overengineer or to shoot for
>    unattainable perfection, but we still try to make sure we will
>    not paint ourselves into an unescapable corner when we later want
>    to extend it.

And there is such a thing as being too cautious, to the point where you walk WAY too slowly for the fear of tumbling, because it happened a few times in the fast, when you were running. But how would you even know that you are being too cautions? If you don't dare to walk a bit faster to find out.

Cheers.

-- 
Felipe Contreras