Paul Mackerras wrote:
> I thought I had replied to this patch; maybe I only thought about it.
> Given that we already have a selector to choose between exact and
> regexp matching, it seems more natural to use that rather than add a
> new selector entry. Arguably the "IgnCase" option should be disabled
> when "adding/removing string" is selected.
Thanks. I think I disagree: "log -G" and "log -S" are different
operations, not variations on the same one.
The description "Find next commit adding/removing string:" very
clearly conveys what "-S" means. Maybe -G would be more clearly
described as "Find next commit changing line that matches regex:" or
"Find next commit changing line containing:"?
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html