[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