On Sep 26, 2010, at 10:24 PM, Junio C Hamano wrote: > Kevin Ballard <kevin@xxxxxx> writes: > >>> At the end-user level, this new option to merge-recursive has exactly the >>> same meaning as existing -M given to "diff" family; people would probably >>> want to see it made available as a synonym to "diff" family as well, no? >> >> You mean so you can type `git diff --rename-score=50% foo`? A reasonable >> suggestion, but then what do we do with -B and -C? It doesn't make much >> sense to give a longer name to only one of the three options. This patch >> was concerned with simply exposing the functionality to the merge >> strategy and doesn't attempt to address the problem of providing long >> names for this trio of options. > > I would call them --break-threshold and --copy-threshold respectively. > > I have been happy without long option names when we originally had only > short names, but some people seem to be able to be more explicit, so... After taking a look at this, it raises another question. -B, -M, and -C all have optional arguments, but the long-form names don't seem to support that. `git diff --rename-threshold= foo` would work, but looks mighty odd, and if I make it support `git diff --rename-threshold foo` that would also work, but the name doesn't seem appropriate without the argument. Should I go ahead and support `git diff --rename-threshold foo` and just live with it looking weird, or do you have a better suggestion? -Kevin Ballard-- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html