aboutgitcodebugslistschat
diff options
context:
space:
mode:
authorDavid Gibson <david@gibson.dropbear.id.au>2022-09-28 14:33:24 +1000
committerStefano Brivio <sbrivio@redhat.com>2022-09-29 12:22:15 +0200
commitfb15259205c006149fc0bedbf4ef135e53c4268e (patch)
tree151bf9630d8f264113fedadd084ad85378b29104
parente2b7d370d08257c14d9288d6b7534d45d57c6816 (diff)
downloadpasst-fb15259205c006149fc0bedbf4ef135e53c4268e.tar
passt-fb15259205c006149fc0bedbf4ef135e53c4268e.tar.gz
passt-fb15259205c006149fc0bedbf4ef135e53c4268e.tar.bz2
passt-fb15259205c006149fc0bedbf4ef135e53c4268e.tar.lz
passt-fb15259205c006149fc0bedbf4ef135e53c4268e.tar.xz
passt-fb15259205c006149fc0bedbf4ef135e53c4268e.tar.zst
passt-fb15259205c006149fc0bedbf4ef135e53c4268e.zip
cppcheck: Use inline suppression for ffsl()
We define our own ffsl() as a weak symbol, in case our C library doesn't include it. On glibc systems which *do* include it, this causes a cppcheck warning because unsurprisingly our version doesn't pick the same argument names. Convert the suppression for this into an inline suppression. Signed-off-by: David Gibson <david@gibson.dropbear.id.au> Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
-rw-r--r--Makefile2
-rw-r--r--util.h1
2 files changed, 1 insertions, 2 deletions
diff --git a/Makefile b/Makefile
index e0933f3..d5bfce4 100644
--- a/Makefile
+++ b/Makefile
@@ -290,7 +290,6 @@ cppcheck: $(SRCS) $(HEADERS)
--suppress=strtokCalled:conf.c --suppress=strtokCalled:qrap.c \
--suppress=localtimeCalled:pcap.c \
--suppress=unusedStructMember:pcap.c \
- --suppress=funcArgNamesDifferent:util.h \
--suppress=unusedStructMember:dhcp.c \
\
--suppress=unmatchedSuppression:conf.c \
@@ -300,6 +299,5 @@ cppcheck: $(SRCS) $(HEADERS)
--suppress=unmatchedSuppression:tcp.c \
--suppress=unmatchedSuppression:udp.c \
--suppress=unmatchedSuppression:util.c \
- --suppress=unmatchedSuppression:util.h \
$(filter -D%,$(FLAGS) $(CFLAGS)) \
.
diff --git a/util.h b/util.h
index 1003303..0c3c994 100644
--- a/util.h
+++ b/util.h
@@ -217,6 +217,7 @@ struct ipv6_opt_hdr {
*/
} __attribute__((packed)); /* required for some archs */
+/* cppcheck-suppress funcArgNamesDifferent */
__attribute__ ((weak)) int ffsl(long int i) { return __builtin_ffsl(i); }
void __openlog(const char *ident, int option, int facility);
void passt_vsyslog(int pri, const char *format, va_list ap);