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

Re: [PATCH] daemon: use strbuf for hostname info

From
Jeff King <peff@peff.net>
Date
Mar 7, 2015, 01:08 UTC
Message-ID
<20150307010832.GB8202@peff.net>
In-Reply-To
<54FA4446.5050103@web.de>
On Sat, Mar 07, 2015 at 01:20:22AM +0100, René Scharfe wrote:
Show 6 quoted lines
> > Not a big deal, but do we want to rename sanitize_client_strbuf to
> > sanitize_client? It only had the unwieldy name to distinguish it from
> > this one.
> 
> A patch would look like this.  The result is shorter, but no win in
> terms of vertical space (number of lines).

IMHO this is an improvement, though whether it is enough to merit the code churn I dunno. So I'm in favor, but don't mind dropping it if others disagree.

-Peff
Previous: René ScharfeNext: René Scharfe
Message 4 of 9 in “daemon: use strbuf for hostname info”
  1. daemon: use strbuf for hostname infoRené Scharfe, Mar 6, 2015
  2. Jeff KingMar 6, 2015
  3. René ScharfeMar 7, 2015
  4. Jeff KingMar 7, 2015
  5. René ScharfeMar 7, 2015
  6. René ScharfeMar 7, 2015
  7. Jeff KingMar 7, 2015
  8. 1/2 daemon: use strbuf for hostname infoRené Scharfe, Mar 7, 2015
  9. 2/2 daemon: deglobalize hostname informationRené Scharfe, Mar 7, 2015

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.