path: root/t/
diff options
authorJunio C Hamano <>2005-05-29 23:54:59 (GMT)
committerLinus Torvalds <>2005-05-30 17:35:49 (GMT)
commitddafa7e93325d45cd4bd950dd8e89ff3188d0250 (patch)
tree61db38734c639c161a70781c9e56a790eb5d8ac0 /t/
parentcad88fdf8d1ebafb5d4d1b92eb243ff86bae740b (diff)
[PATCH] diff-helper: Fix R/C score parsing under -z flag.
The score number that follow R/C status were parsed but the parse pointer was not updated, causing the entire line to become unrecognized. This patch fixes this problem. There was a test missing to catch this breakage, which this commit adds as The diff-raw tests used in related (the same test without -z) and were stricter than necessarily, despite that the comment for the tests said otherwise. This patch also corrects them. The documentation is updated to say that the status can optionally be followed by a number called "score"; it does not have to stay similarity index forever and there is no reason to limit it only to C and R. Signed-off-by: Junio C Hamano <> Signed-off-by: Linus Torvalds <>
Diffstat (limited to 't/')
1 files changed, 13 insertions, 7 deletions
diff --git a/t/ b/t/
index a519855..5ac29d1 100644
--- a/t/
+++ b/t/
@@ -8,11 +8,17 @@ test_description='Same rename detection as t4003 but testing diff-raw.
. ./
+sanitize_diff_raw='s/ '"$_x40"' '"$_x40"' \([A-Z]\)[0-9]* / X X \1# /'
compare_diff_raw () {
# When heuristics are improved, the score numbers would change.
# Ignore them while comparing.
- sed -e 's/ \([CR]\)[0-9]* /\1#/' <"$1" >.tmp-1
- sed -e 's/ \([CR]\)[0-9]* /\1#/' <"$2" >.tmp-2
+ # Also we do not check SHA1 hash generation in this test, which
+ # is a job for
+ sed -e "$sanitize_diff_raw" <"$1" >.tmp-1
+ sed -e "$sanitize_diff_raw" <"$2" >.tmp-2
diff -u .tmp-1 .tmp-2 && rm -f .tmp-1 .tmp-2
@@ -109,11 +115,6 @@ test_expect_success \
'validate output from rename/copy detection (#2)' \
'compare_diff_raw current expected'
-test_expect_success \
- 'prepare work tree once again' \
- 'cat ../../COPYING >COPYING &&
- git-update-cache --add --remove COPYING COPYING.1'
# make sure diff-helper can grok it.
mv expected diff-raw
GIT_DIFF_OPTS=--unified=0 git-diff-helper <diff-raw >current
@@ -151,6 +152,11 @@ test_expect_success \
# anything about rezrov nor COPYING, since the revised again diff-raw
# nows how to say Copy.
+test_expect_success \
+ 'prepare work tree once again' \
+ 'cat ../../COPYING >COPYING &&
+ git-update-cache --add --remove COPYING COPYING.1'
git-diff-cache -C $tree >current
cat >expected <<\EOF
:100644 100644 6ff87c4664981e4397625791c8ea3bbb5f2279a3 0603b3238a076dc6c8022aedc6648fa523a17178 C1234 COPYING COPYING.1