diff options
author | Bo Yang <struggleyb.nku@gmail.com> | 2010-05-06 21:52:28 -0700 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2010-05-07 09:34:28 -0700 |
commit | 1da6175d438a9849db07a68326ee05f291510074 (patch) | |
tree | 1c2cc5a775f5c8f153a7e27e6bbb84a80ef16401 /diffcore.h | |
parent | 9ca5df90615aa3c6b60e1bc8f03db6cae98e816c (diff) | |
download | git-1da6175d438a9849db07a68326ee05f291510074.tar.gz git-1da6175d438a9849db07a68326ee05f291510074.tar.xz |
Make diffcore_std only can run once before a diff_flush
When file renames/copies detection is turned on, the
second diffcore_std will degrade a 'C' pair to a 'R' pair.
And this may happen when we run 'git log --follow' with
hard copies finding. That is, the try_to_follow_renames()
will run diffcore_std to find the copies, and then
'git log' will issue another diffcore_std, which will reduce
'src->rename_used' and recognize this copy as a rename.
This is not what we want.
So, I think we really don't need to run diffcore_std more
than one time.
Signed-off-by: Bo Yang <struggleyb.nku@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'diffcore.h')
-rw-r--r-- | diffcore.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/diffcore.h b/diffcore.h index 5d05deaf6..491bea0b4 100644 --- a/diffcore.h +++ b/diffcore.h @@ -91,11 +91,13 @@ struct diff_queue_struct { struct diff_filepair **queue; int alloc; int nr; + int run; }; #define DIFF_QUEUE_CLEAR(q) \ do { \ (q)->queue = NULL; \ (q)->nr = (q)->alloc = 0; \ + (q)->run = 0; \ } while(0); extern struct diff_queue_struct diff_queued_diff; |