From: Alex Riesen <fork0@t-online.de>
Date: Fri, 12 Jan 2007 20:30:42 GMT
Subject: Re: [PATCH] Speedup recursive by flushing index only once for all entries
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:
> > ...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.