[libc-commits] [libc] [libc][signal] clean up usage of sighandler_t (PR #125745)

Nick Desaulniers via libc-commits libc-commits at lists.llvm.org
Wed Feb 5 15:56:36 PST 2025


================
@@ -10,5 +10,6 @@
 #define LLVM_LIBC_TYPES___SIGHANDLER_T_H
 
 typedef void (*__sighandler_t)(int);
----------------
nickdesaulniers wrote:

This comment was left on a previous version of the patch series. Been fixed already. Reload your browser tab. ;)

That said, I'm doing neither include linux/signal.h nor copy+paste; currently we provide no declaraion of `__sighandler_t` through inclusion of our `<signal.h>` at all.  Perhaps that's problematic for users that really insist on using `__sighandler_t` in their code?

I know your position on making their lives easier...

https://github.com/llvm/llvm-project/pull/125745


More information about the libc-commits mailing list