diff options
| author | Sam McCall <sam.mccall@gmail.com> | 2019-11-15 15:08:16 +0100 |
|---|---|---|
| committer | Sam McCall <sam.mccall@gmail.com> | 2019-11-15 17:32:55 +0100 |
| commit | 713c30b389602eda5c70b696e8c640487cc8b2cb (patch) | |
| tree | 863c99112a69a2b6263754fc594b19e09a092422 /lldb/packages/Python/lldbsuite/test/python_api/interpreter/TestCommandInterpreterAPI.py | |
| parent | ad9fd320091d44d4b8782c28b72a7be21a2bd68d (diff) | |
| download | bcm5719-llvm-713c30b389602eda5c70b696e8c640487cc8b2cb.tar.gz bcm5719-llvm-713c30b389602eda5c70b696e8c640487cc8b2cb.zip | |
[clangd] Don't consider class template params part of constructor name.
Summary:
This is shorter and usually the extra info is noise.
There are cases where the params become type-parameter-0-0 that are hard to fix.
This affects a few features:
- 'name' field in structured hover API (not exposed yet)
- 'name' field in locateSymbolAt (not exposed in LSP)
- 'document/symbol' - the symbol is hierarchically nested in the class
template, or written as foo<t>::foo when defined out-of-line.
Added a test case for hover from https://github.com/clangd/clangd/issues/76.
This patch fixes one field, but no fewer than four others are wrong!
I'll fix them...
Reviewers: hokein
Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, kadircet, usaxena95, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D70308
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/python_api/interpreter/TestCommandInterpreterAPI.py')
0 files changed, 0 insertions, 0 deletions
