This revision was automatically updated to reflect the committed changes. Closed by commit rL346671: [clangd] Don't show all refs results if -name is ambiguous in dexp. (authored by hokein, committed by ). Herald added a subscriber: llvm-commits.
Repository: rL LLVM https://reviews.llvm.org/D54430 Files: clang-tools-extra/trunk/clangd/index/dex/dexp/Dexp.cpp Index: clang-tools-extra/trunk/clangd/index/dex/dexp/Dexp.cpp =================================================================== --- clang-tools-extra/trunk/clangd/index/dex/dexp/Dexp.cpp +++ clang-tools-extra/trunk/clangd/index/dex/dexp/Dexp.cpp @@ -218,6 +218,12 @@ IDs.push_back(*SID); } else { IDs = getSymbolIDsFromIndex(Name, Index); + if (IDs.size() > 1) { + outs() << formatv("The name {0} is ambiguous, found {1} different " + "symbols. Please use id flag to disambiguate.\n", + Name, IDs.size()); + return; + } } RefsRequest RefRequest; RefRequest.IDs.insert(IDs.begin(), IDs.end());
Index: clang-tools-extra/trunk/clangd/index/dex/dexp/Dexp.cpp =================================================================== --- clang-tools-extra/trunk/clangd/index/dex/dexp/Dexp.cpp +++ clang-tools-extra/trunk/clangd/index/dex/dexp/Dexp.cpp @@ -218,6 +218,12 @@ IDs.push_back(*SID); } else { IDs = getSymbolIDsFromIndex(Name, Index); + if (IDs.size() > 1) { + outs() << formatv("The name {0} is ambiguous, found {1} different " + "symbols. Please use id flag to disambiguate.\n", + Name, IDs.size()); + return; + } } RefsRequest RefRequest; RefRequest.IDs.insert(IDs.begin(), IDs.end());
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits