[clang-tools-extra] [clangd] Add quick fix to automatically adds NOLINTNEXTLINE comment (PR #114661)
kadir çetinkaya via cfe-commits
cfe-commits at lists.llvm.org
Fri Dec 6 01:57:22 PST 2024
kadircet wrote:
(sorry for the late reply)
> Just want to confirm that this infrastructure isn't currently being used by anything in Clangd.
we do have some out-of-tree usages, but nothing in the tree, apart from tests, use this infrastructure yet.
> I assume I should extend FeatureModule to add my NOLINT fix feature and add it in ClangMain.cpp conditioning on command line options. Would that be the correct approach?
yes, we basically should have a new feature-module, from that new feature-module we can contribute a tweak. this tweak can look at all the diagnostics in the AST, find the ones that intersect with users selection, and if they're from clang-tidy emit a code-action that inserts `` // NOLINTNEXTLINE(check-name)`.
https://github.com/llvm/llvm-project/pull/114661
More information about the cfe-commits
mailing list