Skip to content

Commit

Permalink
perf(bash-v2): use backslash escape string expansion for tab (#1682)
Browse files Browse the repository at this point in the history
Using a command substitution, i.e. a subshell, with `printf` is
expensive for this purpose. For example `__*_format_comp_descriptions`
is run once for each completion candidate; the expense adds up and
shows when there are a lot of them.
  • Loading branch information
scop authored Apr 28, 2022
1 parent d8184d3 commit 2722a75
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions bash_completionsV2.go
Original file line number Diff line number Diff line change
Expand Up @@ -154,8 +154,7 @@ __%[1]s_handle_completion_types() {
# If the user requested inserting one completion at a time, or all
# completions at once on the command-line we must remove the descriptions.
# https://github.com/spf13/cobra/issues/1508
local tab comp
tab=$(printf '\t')
local tab=$'\t' comp
while IFS='' read -r comp; do
# Strip any description
comp=${comp%%%%$tab*}
Expand All @@ -175,8 +174,7 @@ __%[1]s_handle_completion_types() {
}
__%[1]s_handle_standard_completion_case() {
local tab comp
tab=$(printf '\t')
local tab=$'\t' comp
local longest=0
# Look for the longest completion so that we can format things nicely
Expand Down Expand Up @@ -231,8 +229,7 @@ __%[1]s_handle_special_char()
__%[1]s_format_comp_descriptions()
{
local tab
tab=$(printf '\t')
local tab=$'\t'
local comp="$1"
local longest=$2
Expand Down

0 comments on commit 2722a75

Please sign in to comment.