On Thu, Jul 31, 2008 at 12:05:22AM -0700, Junio C Hamano wrote: > We can probably pick up the result revision parser parsed out of > revs.diffopt, and then tweak "opt" with it, perhaps like this. That is a sensible solution for this option, but I have to wonder: how many other such ineffective options are hiding? How many of them actually have a matching meaning in git-blame? E.g., what does "git blame --name-only" mean? Perhaps we should simply not worry about those ones, as people are unlikely to try using them, and it is easy to say "has no impact, because it doesn't make sense with blame." The truly confusing ones are ones you _expect_ to do something, but don't (like --find-copies-harder). I took a look at implementing a "don't parse the diff options" flag, but it is much larger than that. The revision parser understands a lot of options that don't really make sense for blame (or shortlog), like "--full-diff". So perhaps it is best to just fix this one (which we have actually had a bug report about) and not worry about the rest. -Peff -- 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
