From 0a587335d2a16a48479b10969491a7353d78c25b Mon Sep 17 00:00:00 2001 From: =?utf8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= Date: Tue, 2 Oct 2018 12:26:23 +0200 Subject: [PATCH] journal-verify: add comment and silence LGTM warning --- src/journal/journal-verify.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/journal/journal-verify.c b/src/journal/journal-verify.c index c2f0467..5569ddc 100644 --- a/src/journal/journal-verify.c +++ b/src/journal/journal-verify.c @@ -60,10 +60,11 @@ static void draw_progress(uint64_t p, usec_t *last_usec) { } static uint64_t scale_progress(uint64_t scale, uint64_t p, uint64_t m) { + /* Calculates scale * p / m, but handles m == 0 safely, and saturates. + * Currently all callers use m >= 1, but we keep the check to be defensive. + */ - /* Calculates scale * p / m, but handles m == 0 safely, and saturates */ - - if (p >= m || m == 0) + if (p >= m || m == 0) /* lgtm [cpp/constant-comparison] */ return scale; return scale * p / m; -- 2.7.4