]> git.ipfire.org Git - thirdparty/git.git/commit
commit: simplify code
authorJohannes Schindelin <johannes.schindelin@gmx.de>
Thu, 15 May 2025 13:11:39 +0000 (13:11 +0000)
committerJunio C Hamano <gitster@pobox.com>
Thu, 15 May 2025 20:46:44 +0000 (13:46 -0700)
commit131a8fa8151c95f309241ead33018f30f57ff57c
tree38787c8e0b0ec935fab7275eabb44bdd542e2d68
parent1a8a4971cc6c179c4dd711f4a7f5d7178f4b3ab7
commit: simplify code

The difference of two unsigned integers is defined to be unsigned, and
therefore it is misleading to check whether it is greater than zero
(instead, the more natural way would be to check whether the difference
is zero or not).

Let's instead avoid the subtraction altogether, and compare the two
operands directly, which makes the code more obvious as a side effect.

Pointed out by CodeQL's rule with the ID
`cpp/unsigned-difference-expression-compared-zero`.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/commit.c