[Lldb-commits] [PATCH] D91103: [tooling] Add support for fixits that indicate code will need reformatting
Nathan James via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed Nov 11 03:27:47 PST 2020
njames93 added inline comments.
================
Comment at: clang-tools-extra/clangd/Diagnostics.cpp:636
+ // Filter out any reformat fixits, we don't handle these.
+ // FIXME: Can we?
+ llvm::erase_if(FixIts,
----------------
kadircet wrote:
> in theory yes, as we have access to source manager, we can fetch file contents and create formatted replacements (see `cleanupAndFormat`). but formatting those fixes can imply significant delays on clangd's diagnostic cycles (if there are many of those), that's the reason why we currently don't format fixits.
I mean if clangd is extended to support async code actions for diagnostics instead of just using the inline extension. Having said that, if that were the case, this would probably not be where that support is implemented.
================
Comment at: clang-tools-extra/clangd/Diagnostics.cpp:637
+ // FIXME: Can we?
+ llvm::erase_if(FixIts,
+ [](const FixItHint &Fix) { return Fix.isReformatFixit(); });
----------------
kadircet wrote:
> rather than doing an extra loop, can we just skip those in the for loop below ?
We could, however that would result in messier code further down when trying to build a synthetic message.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91103/new/
https://reviews.llvm.org/D91103
More information about the lldb-commits
mailing list