git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[RFC/PATCH v2 8/8] completion: small optimization

From
Felipe Contreras <felipe.contreras@gmail.com>
Date
Nov 18, 2012, 10:56 UTC
Message-ID
<1353236213-13471-9-git-send-email-felipe.contreras@gmail.com>
In-Reply-To
<1353236213-13471-1-git-send-email-felipe.contreras@gmail.com>

No need to calculate a new $c with a space if we are not going to do anything it with it.

There should be no functional changes, except that a word "foo " with no suffixes can't be matched. But $cur cannot have a space at the end anyway. So it's safe.

Based on the code from SZEDER Gábor.
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 contrib/completion/git-completion.bash | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
index 82ea7b1..e5f862c 100644
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -240,11 +240,11 @@ __gitcomp ()
 		local c i=0 IFS=$' \t\n'
 		for c in $1; do
 			c="$c${4-}"
-			case $c in
-			--*=*|*.) ;;
-			*) c="$c " ;;
-			esac
 			if [[ $c == "$cur_"* ]]; then
+				case $c in
+				--*=*|*.) ;;
+				*) c="$c " ;;
+				esac
 				COMPREPLY[i++]="${2-}$c"
 			fi
 		done
-- 
1.8.0
Previous: Felipe Contreras
Message 9 of 9 in “completion: compgen/compadd cleanups”
  1. 0/8 completion: compgen/compadd cleanupsFelipe Contreras, Nov 18, 2012
  2. 1/8 completion: trivial test improvementFelipe Contreras, Nov 18, 2012
  3. 2/8 completion: get rid of empty COMPREPLY assignmentsFelipe Contreras, Nov 18, 2012
  4. 3/8 completion: add new __gitcompadd helperFelipe Contreras, Nov 18, 2012
  5. 4/8 tmp: completion: add SZEDER's testsFelipe Contreras, Nov 18, 2012
  6. 5/8 completion: get rid of compgenFelipe Contreras, Nov 18, 2012
  7. 6/8 tmp: completion: mark SZEDER's tests as fixedFelipe Contreras, Nov 18, 2012
  8. 7/8 completion: get rid of __gitcomp_1Felipe Contreras, Nov 18, 2012
  9. 8/8 completion: small optimizationFelipe Contreras, Nov 18, 2012

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.