From: Steven Grimm <koreth@midwinter.com>
To: Sam Vilain <sam@vilain.net>
Cc: Git Mailing List <git@vger.kernel.org>
Subject: Re: Directory renames (was Re: blame follows renames, but log doesn't)
Date: Wed, 20 Jun 2007 13:59:45 -0700 [thread overview]
Message-ID: <46799541.6020608@midwinter.com> (raw)
In-Reply-To: <46798B8E.6040103@vilain.net>
Sam Vilain wrote:
> I didn't review your test cases in detail, but they seemed to suffer
> from what I call "over-trivialization"; the heuristic methods don't work
> very well for these non-real-world test cases because they're not long
> enough. Are you confident that these deficiencies are still there with
> longer examples?
>
Those test cases were a demonstration of something I actually ran into
on a real-world project yesterday. The test cases are trivial and short
simply to make them easy to follow in an email message. If you
substitute longer contents for the test files in my example, you will
see the exact same behavior. The real file in question is around 2KB
long, not a monster but presumably long enough that the heuristics
should work.
Also -- though this doesn't happen to be relevant in the case where I
ran into this -- not all files in real-world projects are huge. If the
heuristics break on small test-case files then they will break on small
real-world files too. If nothing else, a real-world project can itself
contain trivial test data (for testing the project, not testing the
version control system) in the form of lots of small files with similar
or identical contents.
-Steve
next prev parent reply other threads:[~2007-06-20 20:59 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-06-19 1:10 blame follows renames, but log doesn't Martin Langhoff
2007-06-19 1:34 ` Sam Vilain
2007-06-19 7:19 ` Theodore Tso
2007-06-19 8:31 ` Martin Langhoff
2007-06-19 8:39 ` Junio C Hamano
2007-06-19 9:54 ` Steven Grimm
2007-06-19 18:28 ` Directory renames (was Re: blame follows renames, but log doesn't) Steven Grimm
2007-06-20 20:18 ` Sam Vilain
2007-06-20 20:59 ` Steven Grimm [this message]
2007-06-20 22:11 ` blame follows renames, but log doesn't Jakub Narebski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=46799541.6020608@midwinter.com \
--to=koreth@midwinter.com \
--cc=git@vger.kernel.org \
--cc=sam@vilain.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).