[PATCH] D61850: Removed superfluous and slightly annoying newlines in run-clang-tidy's output.
Sven Panne via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu May 16 04:46:54 PDT 2019
svenpanne added a comment.
I think `ninja clang-test-tools` is the right target here, and it is happy with the change, which is not very surprising: AFAICT, there is just a single, very simple check for `run-clang-tidy.py`. Anyway, my manual testing with and without `-quiet` shows no problems, either. Furthermore, we have a patched version with the above change in our CI for quite some time, and it works fine. And if I read the git history correctly, the output should be identical to the one before a fix for interleaved output was landed (that fix added the newlines somehow).
Repository:
rCTE Clang Tools Extra
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61850/new/
https://reviews.llvm.org/D61850
More information about the cfe-commits
mailing list