Re: [PATCH v8] diff.c: keep arrow(=) on show_stats()'s shortened filename part to make rename visible

2013-10-22 Thread Yoshioka Tsuneo
after seeing the implementation, anyway :-) # And, my original explanation about the patch might be not enough. Thanks ! --- Tsuneo Yoshioka (吉岡 恒夫) yoshiokatsu...@gmail.com On Oct 22, 2013, at 9:09 PM, Junio C Hamano gits...@pobox.com wrote: Yoshioka Tsuneo yoshiokatsu...@gmail.com writes

Re: [PATCH v8] diff.c: keep arrow(=) on show_stats()'s shortened filename part to make rename visible

2013-10-19 Thread Yoshioka Tsuneo
would think. On Oct 18, 2013, at 1:38 AM, Junio C Hamano gits...@pobox.com wrote: Yoshioka Tsuneo yoshiokatsu...@gmail.com writes: In the [PATCH v7], I changed to keep filename part of suffix to handle above case, but not always keep directory part because I feel totally keeping all part of long

[PATCH v8] diff.c: keep arrow(=) on show_stats()'s shortened filename part to make rename visible

2013-10-18 Thread Yoshioka Tsuneo
git diff -M --stat can detect rename and show renamed file name like foofoofoo = barbarbar. Before this commit, this output is shortened always by omitting left most part like ...foo = barbarbar. So, if the destination filename is too long, source filename putting left or arrow can be totally

Re: [PATCH v6] diff.c: keep arrow(=) on show_stats()'s shortened filename part to make rename visible.

2013-10-18 Thread Yoshioka Tsuneo
...@gmail.com On Oct 18, 2013, at 1:38 AM, Junio C Hamano gits...@pobox.com wrote: Yoshioka Tsuneo yoshiokatsu...@gmail.com writes: In the [PATCH v7], I changed to keep filename part of suffix to handle above case, but not always keep directory part because I feel totally keeping all part

[PATCH] diffcore-rename.c: Estimate filename similarity for rename detection

2013-10-17 Thread Yoshioka Tsuneo
On rename detection like command git diff -M ..., rename is detected based on file similarities. This file similarities are calculated based on the contents of file. And, if the similarities of contents are the same, filename is taken into account. But, the similarity of filename is calculated

[PATCH v2] diffcore-rename.c: Estimate filename similarity for rename detection

2013-10-17 Thread Yoshioka Tsuneo
On rename detection like command git diff -M ..., rename is detected based on file similarities. This file similarities are calculated based on the contents of file. And, if the similarities of contents are the same, filename is taken into account. But, the similarity of filename is calculated

[PATCH v7] diff.c: keep arrow(=) on show_stats()'s shortened filename part to make rename visible

2013-10-17 Thread Yoshioka Tsuneo
git diff -M --stat can detect rename and show renamed file name like foofoofoo = barbarbar. Before this commit, this output is shortened always by omitting left most part like ...foo = barbarbar. So, if the destination filename is too long, source filename putting left or arrow can be totally

Re: [PATCH v6] diff.c: keep arrow(=) on show_stats()'s shortened filename part to make rename visible.

2013-10-17 Thread Yoshioka Tsuneo
And, above may be worse than: ...{...ceDirectory = …ionDirectory}.../nameOfTheFileThatWasMoved I think. Thank you ! --- Tsuneo Yoshioka (吉岡 恒夫) yoshiokatsu...@gmail.com On Oct 17, 2013, at 10:29 PM, Junio C Hamano gits...@pobox.com wrote: Yoshioka Tsuneo yoshiokatsu...@gmail.com writes

Re: [PATCH v5] diff.c: keep arrow(=) on show_stats()'s shortened filename part to make rename visible.

2013-10-16 Thread Yoshioka Tsuneo
Hello Junio, Keshav Thank you very much for your detailed reviewing. I just tried to update the patch and posted as [PATCH v6]. --- Tsuneo Yoshioka (吉岡 恒夫) yoshiokatsu...@gmail.com On Oct 16, 2013, at 1:54 AM, Junio C Hamano gits...@pobox.com wrote: Yoshioka Tsuneo yoshiokatsu...@gmail.com

[PATCH v6] diff.c: keep arrow(=) on show_stats()'s shortened filename part to make rename visible.

2013-10-16 Thread Yoshioka Tsuneo
git diff -M --stat can detect rename and show renamed file name like foofoofoo = barbarbar. Before this commit, this output is shortened always by omitting left most part like ...foo = barbarbar. So, if the destination filename is too long, source filename putting left or arrow can be totally

Re: [PATCH v3] diff.c: keep arrow(=) on show_stats()'s shortened filename part to make rename visible.

2013-10-15 Thread Yoshioka Tsuneo
, Duy Nguyen pclo...@gmail.com wrote: On Sun, Oct 13, 2013 at 3:48 AM, Yoshioka Tsuneo yoshiokatsu...@gmail.com wrote: git diff -M --stat can detect rename and show renamed file name like foofoofoo = barbarbar, but if destination filename is long the line is shortened like ...barbarbar so

[PATCH v5] diff.c: keep arrow(=) on show_stats()'s shortened filename part to make rename visible.

2013-10-15 Thread Yoshioka Tsuneo
git diff -M --stat can detect rename and show renamed file name like foofoofoo = barbarbar. But if destination filename is long, the line is shortened like ...barbarbar so there is no way to know whether the file is renamed or existed in the source commit. Make sure there is always an arrow, like

Re: [PATCH v4] diff.c: keep arrow(=) on show_stats()'s shortened filename part to make rename visible.

2013-10-15 Thread Yoshioka Tsuneo
Hello Felipe Thank you for pointing out the style issue again. I just fixed it and posted as [PATCH v5]. Thanks! --- Tsuneo Yoshioka (吉岡 恒夫) yoshiokatsu...@gmail.com On Oct 15, 2013, at 1:07 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Tue, Oct 15, 2013 at 4:45 AM, Yoshioka

[PATCH v3] diff.c: keep arrow(=) on show_stats()'s shortened filename part to make rename visible.

2013-10-12 Thread Yoshioka Tsuneo
git diff -M --stat can detect rename and show renamed file name like foofoofoo = barbarbar, but if destination filename is long the line is shortened like ...barbarbar so there is no way to know whether the file is renamed or existed in the source commit. This commit makes it visible like ...foo =

[PATCH v3] diff.c: keep arrow(=) on show_stats()'s shortened filename part to make rename visible.

2013-10-12 Thread Yoshioka Tsuneo
git diff -M --stat can detect rename and show renamed file name like foofoofoo = barbarbar, but if destination filename is long the line is shortened like ...barbarbar so there is no way to know whether the file is renamed or existed in the source commit. This commit makes it visible like ...foo =

Re: [spf:guess,mismatch] [PATCH v2] diff.c: keep arrow(=) on show_stats()'s shortened filename part to make rename visible.

2013-10-12 Thread Yoshioka Tsuneo
Hello On Oct 12, 2013, at 8:35 AM, Keshav Kini keshav.k...@gmail.com wrote: Sam Vilain s...@vilain.net writes: On 10/11/2013 06:07 AM, Yoshioka Tsuneo wrote: + prefix_len = ((prefix_len = 0) ? prefix_len : 0); + strncpy(pre_arrow, arrow

[PATCH] diff.c: keep arrow(=) on show_stats()'s shortened filename part to make rename visible.

2013-10-11 Thread Yoshioka Tsuneo
git diff -M --stat can detect rename and show renamed file name like foofoofoo = barbarbar, but if destination filename is long the line is shortened like ...barbarbar so there is no way to know whether the file is renamed or existed in the source commit. This commit makes it visible like ...foo =

[PATCH v2] diff.c: keep arrow(=) on show_stats()'s shortened filename part to make rename visible.

2013-10-11 Thread Yoshioka Tsuneo
git diff -M --stat can detect rename and show renamed file name like foofoofoo = barbarbar, but if destination filename is long the line is shortened like ...barbarbar so there is no way to know whether the file is renamed or existed in the source commit. This commit makes it visible like ...foo =

[PATCH] diffcore-delta: optimize renames and copies detection (git diff -M/-C)

2013-10-09 Thread Yoshioka Tsuneo
diffcore_count_changes() can return -1 when src_copied is greater than delta_limit, without counting all the src_copied. By that, performance of diff -M/-C can be improved. Signed-off-by: Tsuneo Yoshioka yoshiokatsu...@gmail.com --- diffcore-delta.c | 11 --- diffcore-rename.c | 2 +-