Re: diffcore-rename performance mode

[Linus Torvalds <torvalds@linux-foundation.org>]

On Tue, 18 Sep 2007, Jeff King wrote:
> 
> With this patch:
> 
> diff --git a/diffcore-rename.c b/diffcore-rename.c
> index 6bde439..531a844 100644
> --- a/diffcore-rename.c
> +++ b/diffcore-rename.c
> @@ -362,10 +362,7 @@ void diffcore_rename(struct diff_options *options)
>  			m->score = estimate_similarity(one, two,
>  						       minimum_score);
>  			m->name_score = basename_same(one, two);
> -			diff_free_filespec_data(one);
>  		}
> -		/* We do not need the text anymore */
> -		diff_free_filespec_data(two);
>  		dst_cnt++;
>  	}
>  	/* cost matrix sorted by most to least similar pair */
> 
> My 20-minute diff becomes a 2-minute diff.

I think this is a reasonable patch, but only now *after* we limit the 
rename matrix.

The whole reason for freeing the filespec data was to avoid exploding the 
memory usage, but now that we aggressively limit the number of renames in 
flight anyway, I think we can probably remove the code that frees the file 
data in between all the rename similarity analysis.

			Linus