[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