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 /builtin/commit.c | |
parent | 77b1d9f355b44b68b4bb08a8143d13330fe4c562 (diff) |
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 'builtin/commit.c')
0 files changed, 0 insertions, 0 deletions