[bash] Don’t print function definition when checking for existence (#…

…3448) When just checking whether a function is already defined or not, it’s not necessary to print out it’s definition (should it be defined). bash’s `declare` provides the `-F`-option (which implies `-f`), which should give a minor performance improvement Signed-off-by: Christoph Anton Mitterer <[email protected]>
junegunn · Oct 2, 2023 · e833823 · e833823
1 parent ee4ba10
commit e833823
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/shell/completion.bash b/shell/completion.bash
@@ -12,7 +12,7 @@
 if [[ $- =~ i ]]; then
 
 # To use custom commands instead of find, override _fzf_compgen_{path,dir}
-if ! declare -f _fzf_compgen_path > /dev/null; then
+if ! declare -F _fzf_compgen_path > /dev/null; then
   _fzf_compgen_path() {
     echo "$1"
     command find -L "$1" \
@@ -21,7 +21,7 @@ if ! declare -f _fzf_compgen_path > /dev/null; then
   }
 fi
 
-if ! declare -f _fzf_compgen_dir > /dev/null; then
+if ! declare -F _fzf_compgen_dir > /dev/null; then
   _fzf_compgen_dir() {
     command find -L "$1" \
       -name .git -prune -o -name .hg -prune -o -name .svn -prune -o -type d \