summaryrefslogtreecommitdiff
path: root/builtin/commit.c
diff options
context:
space:
mode:
authorEric Sunshine <sunshine@sunshineco.com>2021-12-09 00:11:10 -0500
committerJunio C Hamano <gitster@pobox.com>2021-12-13 10:29:48 -0800
commit03949e33f58223dd2d8465f4dd8042e5e581fcef (patch)
treeb5d3ed0b3bc19d18309a7563e6045442bd2346fc /builtin/commit.c
parent77b1d9f355b44b68b4bb08a8143d13330fe4c562 (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