Re: svn commit: r996581 - in /subversion/trunk/subversion: libsvn_diff/parse-diff.c tests/cmdline/patch_tests.py

On Sat, 18 Sep 2010, Daniel Shahaf wrote:
> "Fewest path components, shortest basename, total filename length".
>
> It's predictable, but it seems a bit arbitrary?

It does seem a bit arbitrary, but I was able to rationalise it as
follows: "fewest path components" is good for choosing between
"subdir/file" and "../../other-branch/dir/subdir/file"; "shortest
basename" is good for choosing between "file.old" and "file", or
between "file" and "file.new"; "total filename length" is good for
choosing between "subdir/file" and "subdir.old/file". You still need a
tiebreaker of last resort after all that.

--apb (Alan Barrett)
Received on 2010-09-18 09:22:36 CEST