BUILD: cli: clear a maybe-unused warning on some older compilers
authorWilly Tarreau <w@1wt.eu>
Sat, 20 Nov 2021 18:17:38 +0000 (19:17 +0100)
committerWilly Tarreau <w@1wt.eu>
Mon, 10 Jan 2022 07:36:40 +0000 (08:36 +0100)
commitac5b14be149ca9f6d8602bf4968aaf4bd351d54f
tree299378bf7e2f0221abeb4dde48a5d09a1d080f92
parent2693881006f32bd68f36aa6eb90ebfdd9217be0e
BUILD: cli: clear a maybe-unused  warning on some older compilers

The SHOW_TOT() and SHOW_AVG() macros used in cli_io_handler_show_activity()
produce a warning on gcc 4.7 on MIPS with threads disabled because the
compiler doesn't know that global.nbthread is necessarily non-null, hence
that at least one iteration is performed. Let's just change the loop for
a do {} while () that lets the compiler know it's always initialized. It
also has the tiny benefit of making the code shorter.

(cherry picked from commit 97b5d07a3e5a33552327bac2e4c9c6a0496f7b5e)
Signed-off-by: Willy Tarreau <w@1wt.eu>
(cherry picked from commit b9aac36d8d8d3bb05c785aa8f6630338d078f64a)
Signed-off-by: Willy Tarreau <w@1wt.eu>
src/cli.c