Re: [PATCH v2 3/3] commit: use prio_queue_replace() in pop_most_recent_commit(),MIME-Version: 1.0

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Aug 3, 2025, 11:12 UTC
Message-ID: <0d3ddfcc-485e-45cb-28e6-62108e6d5a4d@gmx.de>
In-Reply-To: <494e9686-e014-45f8-86d2-8dfcd6630f46@web.de>

Hi René,

On Fri, 18 Jul 2025, René Scharfe wrote:

Show 5 quoted lines

> Optimize pop_most_recent_commit() by adding the first parent using the
> more efficient prio_queue_peek() and prio_queue_replace() instead of
> prio_queue_get() and prio_queue_put().
> 
> [... clipped ...]

I noticed that v2 of this patch not only made it into `next`, but it also introduced a commit subject suffix (likely unintended?):

  [PATCH v2 3/3] commit: use prio_queue_replace() in pop_most_recent_commit(),MIME-Version: 1.0

Maybe that is something you want to look into?

Ciao, Johannes

$ 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.