[PATCH v7 16/49] completion: bash: remove non-append functionality

From: Felipe Contreras <felipe.contreras@gmail.com>
Date: Apr 1, 2023, 08:45 UTC
Message-ID: <20230401084626.304356-17-felipe.contreras@gmail.com>
In-Reply-To: <20230401084626.304356-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.

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 fd2a43de38..7554c3aeb3 100644
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -318,7 +318,7 @@ __gitcomp_direct ()
 {
 	local IFS=$'\n'
 
-	COMPREPLY=($1)
+	COMPREPLY+=($1)
 }
 
 # Similar to __gitcomp_direct, but appends to COMPREPLY instead.
@@ -470,7 +470,6 @@ __gitcomp_nl_append ()
 #    appended.
 __gitcomp_nl ()
 {
-	COMPREPLY=()
 	__gitcomp_nl_append "$@"
 }

-- 
2.33.0