From a9d2b5b088a03972908bdfd2878091b68e8bc4b0 Mon Sep 17 00:00:00 2001 From: NRK Date: Wed, 15 Jun 2022 12:00:14 +0600 Subject: [PATCH] [ci]: suppress some cppcheck warnings cppcheck warns about passing a plain NULL to construct_argv() main.c:254:36: portability: Passing NULL after the last typed argument to a variadic function leads to undefined behaviour. [varFuncNullUB] fidx, fcnt, w, h, z, NULL); in our case, POSIX mandates `NULL` to be defined as `(void *)0` rather than a plain `0` [0]. and void pointers are required to have the same representation and size as char pointers [1]. so this is not an issue we need to worry about. [0]: https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/stddef.h.html#tag_13_47 [1]: https://port70.net/~nsz/c/c99/n1256.html#6.2.5p27 --- .woodpecker/analysis.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.woodpecker/analysis.yml b/.woodpecker/analysis.yml index 926dbbf..9a18b45 100644 --- a/.woodpecker/analysis.yml +++ b/.woodpecker/analysis.yml @@ -14,6 +14,7 @@ pipeline: --force --quiet --inline-suppr --error-exitcode=1 \ --max-ctu-depth=8 -j"$(nproc)" \ $(make OPT_DEP_DEFAULT="$1" dump_cppflags) \ + --suppress=varFuncNullUB \ *.c } run_tidy() {