Re: [PATCH 20/28] cppcheck: Suppress same-value-in-ternary branches warning

On Wed, Sep 28, 2022 at 10:58:06PM +0200, Stefano Brivio wrote:
...
  On Wed, 28 Sep 2022 14:33:31 +1000
 David Gibson <david(a)gibson.dropbear.id.au> wrote:
 
  TIMER_INTERVAL is the minimum of two separately
defined intervals which
 happen to have the same value at present.  This results in an expression
 which has the same value in both branches of a ternary operator, which
 cppcheck warngs about.  This is logically sound in this case, so suppress
 the error (we appear to already have a similar suppression for clang-tidy).
 
 Also add an unmatchedSuppression suppression, since only some cppcheck
 versions complain about this instance.
 
 Signed-off-by: David Gibson <david(a)gibson.dropbear.id.au>
 ---
  passt.c | 1 +
  1 file changed, 1 insertion(+)
 
 diff --git a/passt.c b/passt.c
 index 4796c89..2c4a986 100644
 --- a/passt.c
 +++ b/passt.c
 @@ -305,6 +305,7 @@ int main(int argc, char **argv)
  
  loop:
  	/* NOLINTNEXTLINE(bugprone-branch-clone): intervals can be the same */
 +	/* cppcheck-suppress [duplicateValueTernary, unmatchedSuppression] */ 
 
 Somewhat surprisingly to me, NOLINTNEXTLINE means "next line of code",
 not next line altogether -- given that this still works (I wouldn't
 have even tried). 
Yeah, I wondered about that and came to the same conclusion.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson