Skip to content

Commit

Permalink
Merge branch 'ss/blame-textconv-fake-working-tree'
Browse files Browse the repository at this point in the history
* ss/blame-textconv-fake-working-tree:
  (squash) test for previous
  blame.c: Properly initialize strbuf after calling, textconv_object()

Conflicts:
	t/t8006-blame-textconv.sh
  • Loading branch information
Junio C Hamano committed Nov 1, 2011
2 parents 7406aa2 + 55e7c0a commit b919f84
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 1 deletion.
4 changes: 3 additions & 1 deletion builtin/blame.c
Original file line number Diff line number Diff line change
Expand Up @@ -2113,8 +2113,10 @@ static struct commit *fake_working_tree_commit(struct diff_options *opt,
switch (st.st_mode & S_IFMT) {
case S_IFREG:
if (DIFF_OPT_TST(opt, ALLOW_TEXTCONV) &&
textconv_object(read_from, mode, null_sha1, &buf.buf, &buf_len))
textconv_object(read_from, mode, null_sha1, &buf.buf, &buf_len)) {
buf.alloc = buf_len;
buf.len = buf_len;
}
else if (strbuf_read_file(&buf, read_from, st.st_size) != st.st_size)
die_errno("cannot open or read '%s'", read_from);
break;
Expand Down
11 changes: 11 additions & 0 deletions t/t8006-blame-textconv.sh
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ EOF
chmod +x helper

test_expect_success 'setup ' '
echo "bin: test number 0" >zero.bin &&
echo "bin: test 1" >one.bin &&
echo "bin: test number 2" >two.bin &&
if test_have_prereq SYMLINKS; then
Expand Down Expand Up @@ -43,6 +44,7 @@ test_expect_success 'no filter specified' '

test_expect_success 'setup textconv filters' '
echo "*.bin diff=test" >.gitattributes &&
echo "zero.bin eol=crlf" >>.gitattributes &&
git config diff.test.textconv ./helper &&
git config diff.test.cachetextconv false
'
Expand Down Expand Up @@ -74,6 +76,15 @@ test_expect_success 'blame --textconv going through revisions' '
test_cmp expected result
'

test_expect_success 'blame --textconv with local changes' '
test_when_finished "git checkout zero.bin" &&
printf "bin: updated number 0\015" >zero.bin &&
git blame --textconv zero.bin >blame &&
expect="(Not Committed Yet ....-..-.. ..:..:.. +0000 1)" &&
expect="$expect converted: updated number 0" &&
expr "$(find_blame <blame)" : "^$expect"
'

test_expect_success 'setup +cachetextconv' '
git config diff.test.cachetextconv true
'
Expand Down

0 comments on commit b919f84

Please sign in to comment.