[PATCH v2 12/45] completion: bash: remove non-append functionality
- From
Felipe Contreras <felipe.contreras@gmail.com>
- Date
- Jun 18, 2021, 18:24 UTC
- Message-ID
- <20210618182518.697912-13-felipe.contreras@gmail.com>
- In-Reply-To
- <20210618182518.697912-1-felipe.contreras@gmail.com>
There's no point in setting COMPREPLY only to override it later, and in fact; we don't do that.
Therefore there's no functional difference between __gitcomp_direct() and __gitcomp_direct_append(), since __gitcomp_direct() *always* operates on empty COMPREPLY.
The same goes for __gitcomp_nl().
This patch makes the functionality of append and non-append functions the same.
There should be no functional changes.
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com> --- contrib/completion/git-completion.bash | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash index bede8a7e2a..8f72e0f66b 100644 --- a/contrib/completion/git-completion.bash +++ b/contrib/completion/git-completion.bash @@ -307,7 +307,7 @@ __gitcomp_direct () { local IFS=$'\n' - COMPREPLY=($1) + COMPREPLY+=($1) } # Similar to __gitcomp_direct, but appends to COMPREPLY instead. @@ -459,7 +459,6 @@ __gitcomp_nl_append () # appended. __gitcomp_nl () { - COMPREPLY=() __gitcomp_nl_append "$@" }
-- 2.32.0