[PATCH] D75053: [clangd] Disable ExtractVariable for C
Sam McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 25 01:47:55 PST 2020
sammccall accepted this revision.
sammccall added inline comments.
This revision is now accepted and ready to land.
================
Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractVariable.cpp:453
const ASTContext &Ctx = Inputs.AST->getASTContext();
+ // FIXME: Enable C (and maybe objective-c) once we start spelling types
+ // explicitly instead of making use of auto.
----------------
I think "non-C++" would be clearer, and don't mention obj-c at all.
Formally C++ is independent of whether obj-c is enabled, and that's relevant here.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75053/new/
https://reviews.llvm.org/D75053
More information about the cfe-commits
mailing list