To get properly integration Clang-Tidy with CLion IDE, next things were implemented:
- Preserve Message, FileOffset, FilePath in the clang-tidy output.
- Export all diagnostics, not just the ones with fixes
- Test-cases
Differential D34404
[Clang-Tidy] Preserve Message, FileOffset, FilePath in Clang-Tidy YAML output vladimir.plyashkun on Jun 20 2017, 8:53 AM. Authored by
Details
To get properly integration Clang-Tidy with CLion IDE, next things were implemented:
Diff Detail
Event TimelineComment Actions Please upload a diff with full context (http://llvm.org/docs/Phabricator.html).
Comment Actions
Comment Actions
Comment Actions
Comment Actions A few more nits.
Comment Actions Thanks, Alex for your suggestions. Comment Actions
Comment Actions I tried landing the patch for you, but it doesn't apply cleanly. One reason is that it contains changes to both cfe and clang-tools-extra repos. But even when I apply the patch to the two directories it breaks a bunch of clang-apply-replacements tests: Failing Tests (4): Clang Tools :: clang-apply-replacements/basic.cpp Clang Tools :: clang-apply-replacements/conflict.cpp Clang Tools :: clang-apply-replacements/crlf.cpp Clang Tools :: clang-apply-replacements/format.cpp Please rebase the patch and split it in two (upload a second differential revision for the clang-tools-extra part). Comment Actions I've splitted single revision into two different revisions:
|