[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