[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