diff options
author | SZEDER Gábor <szeder.dev@gmail.com> | 2019-08-13 14:26:47 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2019-08-13 12:31:04 -0700 |
commit | d9ee1e061783be43d10dee996b85bac0f9223ec4 (patch) | |
tree | 6d9424860ed5c955f45c2c8bb91c116eb6393656 /t/t4013/diff.diff-tree_--pretty=oneline_--patch-with-raw_initial | |
parent | completion: use 'sort -u' to deduplicate config variable names (diff) | |
download | tgif-d9ee1e061783be43d10dee996b85bac0f9223ec4.tar.xz |
completion: simplify inner 'case' pattern in __gitcomp()
The second '*' in the '--*=*' pattern of the inner 'case' statement of
the __gitcomp() helper function never matches anything, so let's use
'--*=' instead.
The purpose of that inner case statement is to decide when to append a
trailing space to the listed options and when not. When an option
requires a stuck argument, i.e. '--option=', then the trailing space
should not be added, so the user can continue typing the required
argument right away. That '--*=*' pattern is supposed to match these
options, but for this purpose that second '*' is unnecessary, a '--*='
pattern works just as well. That second '*' would only make a
difference in case of a possible completion word like
'--option=value', but our completion script never passes such a word
to __gitcomp(), because the '--option=' and its 'value' must be
completed separately.
Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t4013/diff.diff-tree_--pretty=oneline_--patch-with-raw_initial')
0 files changed, 0 insertions, 0 deletions