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

Re: [PATCH] commit: introduce set_merge_remote_desc()

From
Jeff King <peff@peff.net>
Date
Aug 13, 2016, 12:24 UTC
Message-ID
<20160813122451.vrqqjjpqicqtz2uj@sigill.intra.peff.net>
In-Reply-To
<57AF0D8E.6040309@web.de>
On Sat, Aug 13, 2016 at 02:07:42PM +0200, René Scharfe wrote:
Show 6 quoted lines
> So let's turn this dish into a full menu:
> 
>   commit: use xstrdup() in get_merge_parent()
>   commit: factor out set_merge_remote_desc()
>   merge-recursive: fix verbose output for multiple base trees
>   commit: use FLEX_ARRAY in struct merge_remote_desc

Very nice. Four patches seems like a lot for such a small fix, but each one is so trivial and easy to understand, I think it's worth it.

They all look good to me.
-Peff
Previous: René Scharfe
Message 8 of 8 in “commit: introduce set_merge_remote_desc()”
  1. commit: introduce set_merge_remote_desc()René Scharfe, Aug 13, 2016
  2. Jeff KingAug 13, 2016
  3. René ScharfeAug 13, 2016
  4. 1/4 commit: use xstrdup() in get_merge_parent()René Scharfe, Aug 13, 2016
  5. 2/4 commit: factor out set_merge_remote_desc()René Scharfe, Aug 13, 2016
  6. 3/4 merge-recursive: fix verbose output for multiple base treesRené Scharfe, Aug 13, 2016
  7. 4/4 commit: use FLEX_ARRAY in struct merge_remote_descRené Scharfe, Aug 13, 2016
  8. Jeff KingAug 13, 2016

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.