Re: [PATCH 02/12] grep: remove redundant regflags assignment under PCRE
- From
Jeff King <peff@peff.net>
- Date
- Apr 11, 2017, 10:10 UTC
- Message-ID
- <20170411101010.fl437hkvidkkx54w@sigill.intra.peff.net>
- In-Reply-To
- <20170408132506.5415-3-avarab@gmail.com>
On Sat, Apr 08, 2017 at 01:24:56PM +0000, Ævar Arnfjörð Bjarmason wrote:
Show 14 quoted lines
> Remove a redundant assignment to the "regflags" variable. This
> variable is only used for POSIX regular expression matching, not when
> the PCRE library is used.
>
> This redundant assignment was added as a result of copy/paste
> programming in commit 84befcd0a4 ("grep: add a grep.patternType
> configuration setting", 2012-08-03). That commit modified already
> working code in commit cca2c172e0 ("git-grep: do not die upon -F/-P
> when grep.extendedRegexp is set.", 2011-05-09) which didn't assign to
> regflags when under PCRE.
>
> Revert back to that behavior, more to reduce "wait this is used under
> PCRE how?" confusion when reading the code, than to to save ourselves
> trivial CPU cycles by removing one assignment.Right, this makes sense. I wondered if there might be an obscure case where we set the flag, but I couldn't find one.
-Peff