]> git.ipfire.org Git - thirdparty/git.git/commit
grep: don't set up a "default" repo for grep
authorMartin Ågren <martin.agren@gmail.com>
Sat, 21 Nov 2020 18:31:07 +0000 (19:31 +0100)
committerJunio C Hamano <gitster@pobox.com>
Sat, 21 Nov 2020 22:50:29 +0000 (14:50 -0800)
commit1d3878799f8260968ea9f6a75a92c4daca1da133
treeb39016c5b5c1baddfc98117bc402b2bbd11a258c
parentfaefdd61ec7c7f6f3c8c9907891465ac9a2a1475
grep: don't set up a "default" repo for grep

`init_grep_defaults()` fills a `static struct grep_opt grep_defaults`.
This struct is then used by `grep_init()` as a blueprint for other such
structs. Notably, `grep_init()` takes a `struct repo *` and assigns it
into the target struct.

As a result, it is unnecessary for us to take a `struct repo *` in
`init_grep_defaults()` as well. We assign it into the default struct and
never look at it again. And in light of how we return early if we have
already set up the default struct, it's not just unnecessary, but is
also a bit confusing: If we are called twice and with different repos,
is it a bug or a feature that we ignore the second repo?

Drop the repo parameter for `init_grep_defaults()`.

Signed-off-by: Martin Ågren <martin.agren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Documentation/MyFirstObjectWalk.txt
builtin/grep.c
builtin/log.c
grep.c
grep.h
revision.c