[PATCH 3/3] igmp: Remove apparently unneeded suppression

cppcheck-2.19.1 complains that the unusedFunction suppression in igmp.c doesn't match. That seems like a cppcheck bug, because the function clearly *is* unused. The function exists because otherwise the compiler fails because "ISO C forbids an empty translation unit". mld.c contains an identical unused definition for the same reason, but without the suppression. It doesn't seem to have caused unusedFunction warnings, so maybe cppcheck counts the non-empty translation unit requirement as a "use" of the function? In any case, since omitting the suppression in mld.c seems to be fine, do the same in igmp.c to stop the complaints. Signed-off-by: David Gibson --- igmp.c | 1 - 1 file changed, 1 deletion(-) diff --git a/igmp.c b/igmp.c index 0e775843..a3971fc6 100644 --- a/igmp.c +++ b/igmp.c @@ -13,5 +13,4 @@ */ /* TO BE IMPLEMENTED */ -/* cppcheck-suppress unusedFunction */ __attribute__((__unused__)) static void unused(void) { } -- 2.52.0