Re: [PATCH] Speedup recursive by flushing index only once for all entries

From: Alex Riesen <fork0@t-online.de>
Date: Jan 12, 2007, 20:30 UTC
Message-ID: <20070112203042.GA8127@steel.home>
In-Reply-To: <7vr6u0t87q.fsf@assigned-by-dhcp.cox.net>

Junio C Hamano, Fri, Jan 12, 2007 19:23:37 +0100:

Show 5 quoted lines

> > ...and it is still used here - however, after the patch *result is
> > uninitialized at this point.
> 
> Very true.  This untested patch should fix it.
>

I had to initialize mrtree of merge() with NULL to reproduce it. Sneaky bastard...

> We could even remove the whole printf(), which I tend to prefer.

I agree. The merges of this kind a rare in comparison to simple ones.

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