diff options
| author |  Pranit Bauva <pranit.bauva@gmail.com> | 2021-02-03 22:54:35 +0100 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2021-02-03 14:52:09 -0800 |
| commit | b7a6f163d6273896e4f846f63dd711be0174522b (patch) | |
| tree | 9edd3ed8cfd11e2da5ef943b51269ea829f73561 /builtin | |
| parent | bisect--helper: retire `--bisect-write` subcommand (diff) | |
| download | tgif-b7a6f163d6273896e4f846f63dd711be0174522b.tar.xz | |
bisect--helper: use `res` instead of return in BISECT_RESET case option
Use `res` variable to store `bisect_reset()` output in BISECT_RESET
case option to make bisect--helper.c more consistent.
Mentored-by: Christian Couder <chriscool@tuxfamily.org>
Signed-off-by: Miriam Rubio <mirucam@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin')
| -rw-r--r-- | builtin/bisect--helper.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/builtin/bisect--helper.c b/builtin/bisect--helper.c index 36b50e3aa8..fd018ddda5 100644 --- a/builtin/bisect--helper.c +++ b/builtin/bisect--helper.c @@ -1047,7 +1047,8 @@ int cmd_bisect__helper(int argc, const char **argv, const char *prefix) case BISECT_RESET: if (argc > 1) return error(_("--bisect-reset requires either no argument or a commit")); - return !!bisect_reset(argc ? argv[0] : NULL); + res = bisect_reset(argc ? argv[0] : NULL); + break; case CHECK_AND_SET_TERMS: if (argc != 3) return error(_("--check-and-set-terms requires 3 arguments")); |