From: Nicolas Pitre <nico@fluxnic.net>
Date: Mon, 11 Jan 2010 22:23:21 GMT
Subject: Re: [PATCH] string-list: remove print_string_list, since it is not used anymore.
Message-ID: <alpine.LFD.2.00.1001111722150.10143@xanadu.home>
In-Reply-To: <7v3a2ccmrg.fsf@alter.siamese.dyndns.org>

On Mon, 11 Jan 2010, Junio C Hamano wrote:

> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> 
> > On Mon, 11 Jan 2010, Thiago Farina wrote:
> >
> >> Signed-off-by: Thiago Farina <tfransosi@gmail.com>
> >> ---
> >
> > It was never used, except for debugging.  Does it hurt you really all that 
> > much?
> 
> Exactly my feeling.
> 
> I think I discarded at least two other patches sent to me to remove this
> "unused" (but obviously meant for debugging) function in the past.  I
> guess we'll keep the function this time, too.

If so many people are so inclined to discard it, then maybe a little 
comment could be added to explain its existance?


Nicolas