Don't wrap functions with themselves

Our weird %-expanding function wrappers around kill et all defined
"--wraps" for the same name.

As it turns out, fish follows that one, and executes the completion
multiple times.

I didn't notice because these tend to be rather quick on linux, but on
macOS that's apparently a real issue.

Fixes #5541.

[ci skip]
This commit is contained in:
Fabian Homborg 2019-01-17 09:44:35 +01:00
parent 1b23814f8b
commit 84339d5636

View file

@ -259,23 +259,23 @@ function __fish_expand_pid_args
end end
end end
function bg --wraps bg function bg
builtin bg (__fish_expand_pid_args $argv) builtin bg (__fish_expand_pid_args $argv)
end end
function fg --wraps fg function fg
builtin fg (__fish_expand_pid_args $argv) builtin fg (__fish_expand_pid_args $argv)
end end
function kill --wraps kill function kill
command kill (__fish_expand_pid_args $argv) command kill (__fish_expand_pid_args $argv)
end end
function wait --wraps wait function wait
builtin wait (__fish_expand_pid_args $argv) builtin wait (__fish_expand_pid_args $argv)
end end
function disown --wraps disown function disown
builtin disown (__fish_expand_pid_args $argv) builtin disown (__fish_expand_pid_args $argv)
end end