Subject: Re: [PATCH 03/11] Use new compress helpers in fast-import

From: Junio C Hamano <[email-masked]>
Date: Sun, 03 Feb 2008 14:53:45 -0800

Marco Costalba <[email-masked]> writes:

> Here is slightly more difficult, in particular
> a xrealloc() has been substituted with a
> free() + xmalloc() to keep the code simple.
>
> Signed-off-by: Marco Costalba <[email-masked]>
> ---
>  fast-import.c |   45 +++++++++++++++------------------------------
>  1 files changed, 15 insertions(+), 30 deletions(-)

I'll let Shawn comment on this.  The realloc() does not seem to
be using the contents in the buffer from the previous round, so
I suspect that a free() followed by an independent alloc() would
be an improvement when the later call uses much larger buffer
than the previous one, but would be a waste if the later one
needs smaller buffer.