[PATCH] D82373: [CodeComplete] Tweak code completion for `typename` and `using`.

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 24 08:04:23 PDT 2020


kadircet added inline comments.


================
Comment at: clang/lib/Sema/SemaCodeComplete.cpp:2058
       Builder.AddChunk(CodeCompletionString::CK_HorizontalSpace);
-      Builder.AddPlaceholderChunk("qualifier");
-      Builder.AddTextChunk("::");
----------------
unfortunately the reasoning above doesn't apply here, as a using declaration *must* have a nested-name-specifier. e.g. `using X;` is not allowed. so please revert this (and following on `using typename` it requires a nested-name-specifier too)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82373/new/

https://reviews.llvm.org/D82373





More information about the cfe-commits mailing list