Re: [PATCH 5/6] git-sh-setup: remove unused sane_egrep() function
From: Junio C Hamano <hidden>
Date: 2021-10-22 00:27:33
Ævar Arnfjörð Bjarmason [off-list ref] writes:
The is_zero_oid() function in git-submodule.sh has not been used since e83e3333b57 (submodule: port submodule subcommand 'summary' from shell to C, 2020-08-13), so we can remove it, and the sane_egrep() function, dead is_zero_oid() was the only function which still referenced it. Unlike some other functions in git-sh-setup.sh, this function has not been documented in git-sh-setup(1), so per [1] it should be OK to remove it. I'm still unclear about the future of some of the other functions[2], but any questions in that area should not apply here. 1. https://lore.kernel.org/git/xmqqr1dtgnn8.fsf@gitster.g/ (local) 1. https://lore.kernel.org/git/87tuiwjfvi.fsf@evledraar.gmail.com/ (local)
Don't force readers to go outside of "git log" stream they are reading. Instead, summarize the argument that you base your decision on for readers of your proposed commit log message.
quoted hunk
Signed-off-by: Ævar Arnfjörð Bjarmason <redacted> --- git-sh-setup.sh | 4 ---- 1 file changed, 4 deletions(-)diff --git a/git-sh-setup.sh b/git-sh-setup.sh index cee053cdc38..758520f5a61 100644 --- a/git-sh-setup.sh +++ b/git-sh-setup.sh@@ -177,10 +177,6 @@ sane_grep () { GREP_OPTIONS= LC_ALL=C grep @@SANE_TEXT_GREP@@ "$@" } -sane_egrep () { - GREP_OPTIONS= LC_ALL=C egrep @@SANE_TEXT_GREP@@ "$@" -} - is_bare_repository () { git rev-parse --is-bare-repository }