Re: [PATCH] [clang-tidy] ensure run-clang-tidy reports children killed by signals

2021-02-20 Thread Ian Campbell via cfe-commits
Hi Alexander, Would it be better if I signed up for Phabricator and sent this there? I was trying to avoid signing up to yet another service if it wasn't strictly needed but happy to do it if that's what is expected. Thanks, Ian On Thu, 2021-02-11 at 10:22 +, Ian Campbell wrote: If a clang-t

[PATCH] [clang-tidy] ensure run-clang-tidy reports children killed by signals

2021-02-11 Thread Ian Campbell via cfe-commits
If a clang-tidy child process exits with a signal then run-clang-tidy will exit with an error but there is no hint why in the output, since the clang-tidy doesn't log anything and may not even have had the opportunity to do so depending on the signal used. `subprocess.CompletedProcess.returncode`