2009/3/3 Jeff King <peff@xxxxxxxx>: > On Tue, Mar 03, 2009 at 04:12:30PM +0000, John Tapsell wrote: > >> > To fix it we: >> > >> > 1. use "introduce or remove an instance of" instead of >> > "contain" >> >> I would read this to mean that it doesn't include modifying a line >> containing that string. But I also know that underneath the hood, a >> change is a remove then an addition, so I would be confused :) >> >> What about saying "modifies" rather than "contain" ? > > I'm confused. It _doesn't_ include modifying a line containing the > string. In which case it has done its job. But your "but" after that > is what leaves me confused. You thought it would mean that, but you > don't due to some other knowledge, which is leading you down the wrong > path? Yes, it would seem that I was also confused as to what -S means. It doesn't mean what I thought it meant :-) > I was trying to get away with a short and sweet description. But the > behavior is basically (with a few optimizations): > > if count(a, string) != count(b, string) then > it is interesting > > which is unambiguous, but it takes a second to realize the implications. > > -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
