diff options
| author | Sam McCall <sam.mccall@gmail.com> | 2019-11-15 15:46:17 +0100 |
|---|---|---|
| committer | Sam McCall <sam.mccall@gmail.com> | 2019-11-15 17:34:22 +0100 |
| commit | c9081968ead183ee1df824f7b96fcafcfcbe57cd (patch) | |
| tree | d8f12a3fc18e65faf2d4f36f39cc1631884de355 | |
| parent | f018246c20481d222af4bab1868e8903c35c73d2 (diff) | |
| download | bcm5719-llvm-c9081968ead183ee1df824f7b96fcafcfcbe57cd.tar.gz bcm5719-llvm-c9081968ead183ee1df824f7b96fcafcfcbe57cd.zip | |
[clangd] Fix typo in symbol kind conversion
Reviewers: hokein
Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, kadircet, usaxena95, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D70309
| -rw-r--r-- | clang-tools-extra/clangd/Protocol.cpp | 2 | ||||
| -rw-r--r-- | clang-tools-extra/clangd/unittests/FindSymbolsTests.cpp | 12 | ||||
| -rw-r--r-- | clang-tools-extra/clangd/unittests/XRefsTests.cpp | 2 |
3 files changed, 8 insertions, 8 deletions
diff --git a/clang-tools-extra/clangd/Protocol.cpp b/clang-tools-extra/clangd/Protocol.cpp index 9b0fbc5b2f3..bdf284dc502 100644 --- a/clang-tools-extra/clangd/Protocol.cpp +++ b/clang-tools-extra/clangd/Protocol.cpp @@ -257,7 +257,7 @@ SymbolKind indexSymbolKindToSymbolKind(index::SymbolKind Kind) { return SymbolKind::Property; case index::SymbolKind::Constructor: case index::SymbolKind::Destructor: - return SymbolKind::Method; + return SymbolKind::Constructor; case index::SymbolKind::ConversionFunction: return SymbolKind::Function; case index::SymbolKind::Parameter: diff --git a/clang-tools-extra/clangd/unittests/FindSymbolsTests.cpp b/clang-tools-extra/clangd/unittests/FindSymbolsTests.cpp index 8d463362634..8eebb190eda 100644 --- a/clang-tools-extra/clangd/unittests/FindSymbolsTests.cpp +++ b/clang-tools-extra/clangd/unittests/FindSymbolsTests.cpp @@ -390,16 +390,16 @@ TEST_F(DocumentSymbolsTest, BasicSymbols) { ElementsAreArray( {AllOf(WithName("Foo"), WithKind(SymbolKind::Class), Children()), AllOf(WithName("Foo"), WithKind(SymbolKind::Class), - Children(AllOf(WithName("Foo"), WithKind(SymbolKind::Method), - Children()), - AllOf(WithName("Foo"), WithKind(SymbolKind::Method), - Children()), + Children(AllOf(WithName("Foo"), + WithKind(SymbolKind::Constructor), Children()), + AllOf(WithName("Foo"), + WithKind(SymbolKind::Constructor), Children()), AllOf(WithName("f"), WithKind(SymbolKind::Method), Children()), AllOf(WithName("operator="), WithKind(SymbolKind::Method), Children()), - AllOf(WithName("~Foo"), WithKind(SymbolKind::Method), - Children()), + AllOf(WithName("~Foo"), + WithKind(SymbolKind::Constructor), Children()), AllOf(WithName("Nested"), WithKind(SymbolKind::Class), Children(AllOf(WithName("f"), WithKind(SymbolKind::Method), diff --git a/clang-tools-extra/clangd/unittests/XRefsTests.cpp b/clang-tools-extra/clangd/unittests/XRefsTests.cpp index e896096b7ef..15f94e98677 100644 --- a/clang-tools-extra/clangd/unittests/XRefsTests.cpp +++ b/clang-tools-extra/clangd/unittests/XRefsTests.cpp @@ -917,7 +917,7 @@ void foo())cpp"; HI.NamespaceScope = ""; HI.Name = "X"; HI.LocalScope = "X::"; // FIXME: Should be X<T *>:: - HI.Kind = SymbolKind::Method; // FIXME: Should be Constructor + HI.Kind = SymbolKind::Constructor; HI.Type = "void ()"; // FIXME: Should be None HI.ReturnType = "void"; // FIXME: Should be None or X<T*> HI.Definition = "X<type - parameter - 0 - 0 *>()"; // FIXME: --> X() |
