| Message ID | 20230803-nolibc-warnings-v3-0-bcc1a096ae02@weissschuh.net (mailing list archive) |
|---|---|
| Headers | show |
| Series | tools/nolibc: enable compiler warnings | expand |
On Thu, Aug 03, 2023 at 09:28:44AM +0200, Thomas Weißschuh wrote: > To help the developers to avoid mistakes and keep the code smaller let's > enable compiler warnings. All the series looks good, I've now queued it, thanks! > I stuck with __attribute__((unused)) over __maybe_unused in > nolibc-test.c for consistency with nolibc proper. > If we want to add a define it needs to be added twice once for nolibc > proper and once for nolibc-test otherwise libc-test wouldn't build > anymore. I tend to prefer to avoid spreading macros in nolibc itself unless strictly necessary as we'd need to put them under a "nolibc" namespace to avoid a risk of clash, and it becomes less interesting in terms of number of characters saved per line when everything is prefixed with "nolibc_" or so. It's convenient however when there are multiple choices to be replicated at multiple places. So let's keep it like this for now. Cheers, Willy
To help the developers to avoid mistakes and keep the code smaller let's enable compiler warnings. I stuck with __attribute__((unused)) over __maybe_unused in nolibc-test.c for consistency with nolibc proper. If we want to add a define it needs to be added twice once for nolibc proper and once for nolibc-test otherwise libc-test wouldn't build anymore. Signed-off-by: Thomas Weißschuh <linux@weissschuh.net> --- Changes in v3: - Make getpagesize() return "int" - Simplify validation of read() return value - Don't make functions static that are to be used as breakpoints - Drop -s from LDFLAGS - Use proper types for read()/write() return values - Fix unused parameter warnings in new setvbuf() - Link to v2: https://lore.kernel.org/r/20230801-nolibc-warnings-v2-0-1ba5ca57bd9b@weissschuh.net Changes in v2: - Don't drop unused test helpers, mark them as __attribute__((unused)) - Make some function in nolibc-test static - Also handle -W and -Wextra - Link to v1: https://lore.kernel.org/r/20230731-nolibc-warnings-v1-0-74973d2a52d7@weissschuh.net --- Thomas Weißschuh (14): tools/nolibc: drop unused variables tools/nolibc: fix return type of getpagesize() tools/nolibc: setvbuf: avoid unused parameter warnings tools/nolibc: sys: avoid implicit sign cast tools/nolibc: stdint: use int for size_t on 32bit selftests/nolibc: drop unused variables selftests/nolibc: mark test helpers as potentially unused selftests/nolibc: make functions static if possible selftests/nolibc: avoid unused parameter warnings selftests/nolibc: avoid sign-compare warnings selftests/nolibc: use correct return type for read() and write() selftests/nolibc: prevent out of bounds access in expect_vfprintf selftests/nolibc: don't strip nolibc-test selftests/nolibc: enable compiler warnings tools/include/nolibc/stdint.h | 4 + tools/include/nolibc/stdio.h | 5 +- tools/include/nolibc/sys.h | 7 +- tools/testing/selftests/nolibc/Makefile | 4 +- tools/testing/selftests/nolibc/nolibc-test.c | 111 ++++++++++++++++----------- 5 files changed, 80 insertions(+), 51 deletions(-) --- base-commit: bc87f9562af7b2b4cb07dcaceccfafcf05edaff8 change-id: 20230731-nolibc-warnings-c6e47284ac03 Best regards,