nridge added a comment.

I understood https://github.com/clangd/clangd/issues/1344 as being about 
`workspace/symbol` rather than `textDocument/documentSymbol`, though I see now 
that both are affected.

The analysis is a bit different for the two, though: unlike `DocumentSymbol`, 
the result types for `workspace/symbol` (LSP specifies two options, 
`SymbolInformation` or `WorkspaceSymbol`) do not have a `detail` field. So, for 
`workspace/symbol`, the issue would require a fix on the clangd side (or a 
change to LSP, I guess).

@sammccall, would you support adding the signature to the symbol name for 
`workspace/symbol`?


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

https://reviews.llvm.org/D157080

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to