refactor: normalize unset positions of call-once functions

* bash_completion (_comp_dequote__initialize): The initialization
  function "_comp_dequote__initialize" has been unset itself at the
  beginning, but other similar call-once functions unset themselves at
  the end of the functions.  We make "_comp_dequote__initialize" unset
  itself at the end for consistency with the other call-once
  functions.

* completions/vncviewer (_comp_cmd_vncviewer__bootstrap): This
  function wants to return the exit status of the dispatched function,
  so we unset the function just before the processing of the
  dispatched function, following an existing function
  "_comp_cmd_makepkg__bootstrap".

Author: akinomyoga

bash_completion

@@ -180,10 +180,10 @@ _comp_quote()
 # shellcheck disable=SC1003
 _comp_dequote__initialize()
 {
-    unset -f "$FUNCNAME"
     local regex_param='\$([_a-zA-Z][_a-zA-Z0-9]*|[-*@#?$!0-9_])|\$\{[!#]?([_a-zA-Z][_a-zA-Z0-9]*(\[([0-9]+|[*@])\])?|[-*@#?$!0-9_])\}'
     local regex_quoted='\\.|'\''[^'\'']*'\''|\$?"([^\"$`!]|'$regex_param'|\\.)*"|\$'\''([^\'\'']|\\.)*'\'''
     _comp_dequote__regex_safe_word='^([^\'\''"$`;&|<>()!]|'$regex_quoted'|'$regex_param')*$'
+    unset -f "$FUNCNAME"
 }
 _comp_dequote__initialize
 

completions/vncviewer

@@ -11,9 +11,9 @@ _comp_cmd_vncviewer__bootstrap()
     esac
 
     # Install real completion for subsequent completions
+    unset -f "$FUNCNAME"
     complete -F $fname vncviewer
     $fname "$@" # Generate completions once for now
-    unset -f "$FUNCNAME"
 } &&
     complete -F _comp_cmd_vncviewer__bootstrap vncviewer