]> git.ipfire.org Git - thirdparty/git.git/commit
t/perf: avoid unnecessary test_export() recursion
authorEric Sunshine <sunshine@sunshineco.com>
Sun, 20 Dec 2020 21:27:40 +0000 (16:27 -0500)
committerJunio C Hamano <gitster@pobox.com>
Tue, 22 Dec 2020 21:45:36 +0000 (13:45 -0800)
commit5bc12c11ccdd3126cf04cf08cf59bf3c204f7f2a
treed6fb85ac6b3f0e8f8e4ed500a2d1e4db3a3e6e0e
parentf4698738f9dbe733e64f968fff95a9a4f881431e
t/perf: avoid unnecessary test_export() recursion

test_export() has been self-recursive since its inception even though a
simple for-loop would have served just as well to append its arguments
to the `test_export_` variable separated by the pipe character "|".
Recently `test_export_` was changed instead to a space-separated list of
tokens to be exported, an operation which can be accomplished via a
single simple assignment, with no need for looping or recursion.
Therefore, simplify the implementation.

While at it, take advantage of the fact that variable names to be
exported are shell identifiers, thus won't be composed of special
characters or whitespace, thus simple a `$*` can be used rather than
magical `"$@"`.

Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/perf/perf-lib.sh