[PATCH] D59487: [Tooling] Add more scope specifiers until spelling is not ambiguous.
Kadir Cetinkaya via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 18 09:01:00 PDT 2019
kadircet accepted this revision.
kadircet added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: lib/Tooling/Core/Lookup.cpp:165
+ if (UnspelledScopes.empty()) {
+ Disambiguated = "::" + Disambiguated;
+ } else {
----------------
maybe return or break afterwards?
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59487/new/
https://reviews.llvm.org/D59487
More information about the cfe-commits
mailing list