From: Nguyen Thai Ngoc Duy <pclouds@gmail.com>
Date: Sat, 08 Nov 2008 04:01:20 GMT
Subject: Re: [PATCH 1/2] diffcore-rename: support rename cache
Message-ID: <fcaeb9bf0811072001o6df7ae00k1b1bffaadf75d3a1@mail.gmail.com>
In-Reply-To: <7vy6zvdt25.fsf@gitster.siamese.dyndns.org>

On 11/8/08, Junio C Hamano <gitster@pobox.com> wrote:
> Yann Dirson <ydirson@altern.org> writes:
>
>  > On Fri, Nov 07, 2008 at 09:35:32PM +0700, Nguy???n Thái Ng???c Duy wrote:
>  >> This patch teaches diffcore_rename() to look into
>  >> $GIT_DIR/rename-cache and make use of it to recreate diff_filepair.
>  >> With proper cache, there should be no available entry for estimation
>  >> after exact matching.
>  >
>  > This is something I have thought about in the past, good to see that
>  > implemented :)
>  >
>  >> Rename caching is per commit. I don't think abitrary tree-tree caching
>  >> is worth it.
>  >
>  > That could be a nice complement to my directory-rename patch.
>
>
> Has anybody thought about interaction between that caching and pathspec
>  limited operation?
>

I didn't. But I think all out-of-pathspec diff pairs are removed
before it reaches diffcore_rename() so the cache has nothing to do
with it (except it still loads full cache for a commit).
-- 
Duy