Jeff King <> writes:

>> +    } else if (!strcmp(arg, "--perl-regexp") || !strcmp(arg, "-P")) {
>> +            grep_set_pattern_type_option(GREP_PATTERN_TYPE_PCRE, 
>> &revs->grep_filter);
> Do we want to yield short-and-sweet "-P" to perl-regexp? git-grep does
> so to match GNU grep, but we are not matching anything here (except
> ourselves in git-grep). I'd think most people who use it regularly would
> just set grep.patternType.

My instinct always is that we should not to add short-and-sweet one
letter option until it is known to be necessary and useful; the
above was me typing without thinkng.  And I agree grep.patternType
should be sufficient.

To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to
More majordomo info at

Reply via email to