Re: tools: Use fputc() calls in some functions

Markus Elfring <[email protected]> Fri, 7 Aug 2026 08:33:08 +0200
Newsgroups dev.linux.lists.virtualization,dev.linux.lists.acpica-devel,dev.linux.lists.llvm,dev.linux.lists.mptcp,dev.linux.lists.oe-kbuild-all,org.infradead.lists.linux-arm-kernel,org.kernel.vger.bpf,org.kernel.vger.kernel-janitors,org.kernel.vger.kvm,org.kernel.vger.linux-acpi,org.kernel.vger.linux-gpio,org.kernel.vger.linux-kernel,org.kernel.vger.linux-kselftest,org.kernel.vger.linux-perf-users,org.kernel.vger.linux-pm,org.kernel.vger.linux-trace-kernel,org.kernel.vger.netdev,org.kernel.vger.platform-driver-x86,org.kvack.linux-mm,org.ozlabs.lists.linuxppc-dev
Message-ID <[email protected]> 
…
> All errors (new ones prefixed by >>):
> 
>>> builtin-check.c:300:8: error: incompatible pointer to integer conversion passing 'FILE *' (aka 'struct _IO_FILE *') to parameter of type 'int' [-Wint-conversion]
>      300 |         fputc(stderr, '\n');
>          |               ^~~~~~
>    /usr/include/stdio.h:155:16: note: expanded from macro 'stderr'
>      155 | #define stderr stderr
>          |                ^~~~~~
>    /usr/include/stdio.h:611:23: note: passing argument to parameter '__c' here
>      611 | extern int fputc (int __c, FILE *__stream) __nonnull ((2));
>          |                       ^…

Would you become interested to take another look at a corrected source code transformation approach?

Improving output for single characters (with SmPL)?
https://lore.kernel.org/cocci/[email protected]/
https://sympa.inria.fr/sympa/arc/cocci/2026-07/msg00055.html
https://lkml.org/lkml/2026/7/23/2205

Can similar adjustments become more helpful for further software areas?

Regards,
Markus