git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] string-list: remove print_string_list, since it is not used anymore.

From
Nicolas Pitre <nico@fluxnic.net>
Date
Jan 11, 2010, 22:23 UTC
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:
Show 15 quoted lines
> 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
Previous: Junio C HamanoNext: Thiago Farina
Message 4 of 6 in “string-list: remove print_string_list, since it is not used anymore.”
  1. string-list: remove print_string_list, since it is not used anymore.Thiago Farina, Jan 11, 2010
  2. Johannes SchindelinJan 11, 2010
  3. Junio C HamanoJan 11, 2010
  4. Nicolas PitreJan 11, 2010
  5. Thiago FarinaJan 11, 2010
  6. Junio C HamanoJan 11, 2010

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.