From: Ramkumar Ramachandra <artagnon@gmail.com>
Date: Sun, 23 Jun 2013 08:37:36 GMT
Subject: Re: [PATCH 05/16] remote: remove dead code in read_branches_file()
Message-ID: <CALkWK0=pgivbq0D3Q29H4+RM77s67EALharbVCwAGFqjBPQgwQ@mail.gmail.com>
In-Reply-To: <7vhagpfdwp.fsf@alter.siamese.dyndns.org>

Junio C Hamano wrote:
>>> Reminds me of the strategy to deprecate functionality in X (cf.
>>> http://lwn.net/Articles/536520/) ;-)
>>
>> Leaving dead code around to confuse readers? :\
>
> We broke the use case to access jgarzik/netdev-2.6 only by having
> jgarzik remote accidentally, and waited for quite a while (since
> early 2008 until now) to see if anobody screamed.  Nobody did.  We
> now know we can remove that feature.

Did we _not_ leave dead code around to confuse readers (no comment or
log message indicating intent), or am I missing something?

> That is exactly what Keith is describing, isn't it?

Wasn't that a joke?