diff options
author | Eric Sunshine <sunshine@sunshineco.com> | 2021-12-09 00:11:10 -0500 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2021-12-13 10:29:48 -0800 |
commit | 03949e33f58223dd2d8465f4dd8042e5e581fcef (patch) | |
tree | b5d3ed0b3bc19d18309a7563e6045442bd2346fc /t/lib-pack.sh | |
parent | tests: apply modern idiom for signaling test failure (diff) | |
download | tgif-03949e33f58223dd2d8465f4dd8042e5e581fcef.tar.xz |
tests: apply modern idiom for exiting loop upon failure
Rather than maintaining a flag indicating a failure within a loop and
aborting the test when the loop ends if the flag is set, modern practice
is to signal the failure immediately by exiting the loop early via
`return 1` (or `exit 1` if inside a subshell). Simplify these loops by
following the modern idiom.
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Reviewed-by: Elijah Newren <newren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/lib-pack.sh')
0 files changed, 0 insertions, 0 deletions