From: Phillip Wood <phillip.wood@dunelm.org.uk>
Date: Wed, 15 Jul 2026 15:21:58 +0000 (+0100)
Subject: sequencer: remove unnecessary "or" in pick_one_commit()
X-Git-Url: http://git.ipfire.org/gitweb/?a=commitdiff_plain;h=6bb740dffb908a670e52b88f309f521f6d29a912;p=thirdparty%2Fgit.git

sequencer: remove unnecessary "or" in pick_one_commit()

If error_with_patch(..., res, ...) succeeds then it returns "res", if
it fails then it returns -1. This means that or-ing the return value
with "res" is pointless as the result is the same as the return value.

Signed-off-by: Phillip Wood <phillip.wood@dunelm.org.uk>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
---

diff --git a/sequencer.c b/sequencer.c
index 1db844100a..70e12eab0e 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -5007,9 +5007,8 @@ static int pick_one_commit(struct repository *r,
 		      oideq(&opts->squash_onto, &oid))))
 			to_amend = 1;
 
-		return res | error_with_patch(r, item->commit,
-					      arg, item->arg_len, opts,
-					      res, to_amend);
+		return error_with_patch(r, item->commit, arg, item->arg_len,
+					opts, res, to_amend);
 	}
 	return res;
 }