[PATCH] D69181: [clang-tidy] Adding misc-signal-terminated-thread check
Kocsis Ábel via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Oct 27 16:52:44 PDT 2019
abelkocsis marked 3 inline comments as done.
abelkocsis added inline comments.
================
Comment at: clang-tools-extra/docs/clang-tidy/checks/misc-bad-signal-to-kill-thread.rst:8
+``SIGTERM`` signal and the signal kills the entire process, not just the
+individual thread. Use any signal except ``SIGTERM`` or ``SIGKILL``.
+
----------------
aaron.ballman wrote:
> Why does the check not look for `SIGKILL` as well as `SIGTERM`?
In my opinion `SIGKILL` is used only in cases when the programmer wants the entire process to terminate. I updated the docs not mentioning `SIGKILL`.
Repository:
rCTE Clang Tools Extra
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69181/new/
https://reviews.llvm.org/D69181
More information about the cfe-commits
mailing list