Re: [PATCH 1/3] show-ref: remove unused custom handling of -h
- From
René Scharfe <l.s.r@web.de>
- Date
- Sep 24, 2021, 20:53 UTC
- Message-ID
- <b0b1c64d-9b53-a64f-73db-b8552c7b66a2@web.de>
- In-Reply-To
- <xmqq8rzlu697.fsf@gitster.g>
Am 24.09.21 um 21:24 schrieb Junio C Hamano:
Show 22 quoted lines
> Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes: > >> Since b92891f9783 (parseopt: add PARSE_OPT_NO_INTERNAL_HELP, >> 2009-03-08) parse_options() has handled "-h" unless told not to, so >> when show-ref was migrated to parse_options() in >> 69932bc6117 (show-ref: migrate to parse-options, 2009-06-20) the >> custom "-h" handling that was retained did nothing. >> >> The option was then hidden in e62b3935056 (Show usage string for 'git >> show-ref -h', 2009-11-09), but that OPT_BOOLEAN didn't do >> anything. Let's just remove this dead code. >> >> Reported-by: Ignacy Gawedzki <ignacy.gawedzki@green-communications.fr> >> Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> >> --- >> builtin/show-ref.c | 2 -- >> 1 file changed, 2 deletions(-) > > While this is a fine no-op, I am not sure this "fixes" complaint > in Ignacy's report. "git show-ref -h" would (and should) show the > short-help, no? >
It would, but -h is not a no-op without this patch. The option is equivalent to --heads as long as it's not the only argument. E.g. it has an effect in "git show-ref -h HEA." or "git show-ref -hh".
René