From b8e3b2f33994de25c068021e388dbef72a2cdc4a Mon Sep 17 00:00:00 2001 From: Miriam Rubio Date: Mon, 17 Feb 2020 09:40:30 +0100 Subject: bisect: use the standard 'if (!var)' way to check for 0 Instead of using 'var == 0' in an if condition, let's use '!var' and make 'bisect.c' more consistent with the rest of the code. Mentored-by: Christian Couder Signed-off-by: Miriam Rubio Signed-off-by: Junio C Hamano diff --git a/bisect.c b/bisect.c index e81c91d..83cb5b3 100644 --- a/bisect.c +++ b/bisect.c @@ -572,7 +572,7 @@ static int sqrti(int val) { float d, x = val; - if (val == 0) + if (!val) return 0; do { @@ -869,7 +869,7 @@ static void check_good_are_ancestors_of_bad(struct repository *r, goto done; /* Bisecting with no good rev is ok. */ - if (good_revs.nr == 0) + if (!good_revs.nr) goto done; /* Check if all good revs are ancestor of the bad rev. */ -- cgit v0.10.2-6-g49f6