[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
Mon May 13 11:56:32 PDT 2019
svenpanne added a comment.
In D61850#1500298 <https://reviews.llvm.org/D61850#1500298>, @JonasToth wrote:
> [...] please ensure that e.g. clang-analyzer-* output looks proper as well and `-quiet` does, too.
OK, are there any (unit) tests which I should run? If yes, how exactly? I have a successfully built a fork of https://github.com/llvm/llvm-project via ninja locally, but the documentation for the various subprojects is still a bit hard to find to me. Is there a ninja target for `clang-tidy`-related tests? Any hints to get started would be highly appreciated.
This and the other output-related patches are my first submissions here, and they are intentionally simple to get the workflow right. My real intention is trying to improve/fix the `readability-identifier-naming` check/fixer, which still has a few issues.
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