On Sep 27, 2010, at 4:53 PM, Jonathan Nieder wrote: > Kevin Ballard wrote: > >> 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. > [...] >> if I make it support `git diff >> --rename-threshold foo` that would also work, but the name doesn't >> seem appropriate without the argument. > > Right --- with merge-recursive the argument doesn't need to be > optional, but with git diff it does. > > How about > > --detect-renames=<threshold> > --detect-copies=<threshold> > --detect-rewrites=<threshold>/<threshold> Good timing, I just sent out a patch that does almost exactly this, though I went with --break-rewrites instead of --detect-rewrites. -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