Page MenuHomePhabricator

[clangd] Add doxygen parsing for Hover
AbandonedPublic

Authored by tom-anders on Aug 14 2022, 3:29 AM.

Details

Summary

This adds a new class SymbolDocumentation that does basic doxygen
parsing. Currently, it's only consumed by Hover, but in future
patches we'll probably also want to use it in CodeComplete and
SignatureHelp.

Diff Detail

Event Timeline

tom-anders created this revision.Aug 14 2022, 3:29 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 14 2022, 3:29 AM
tom-anders published this revision for review.Aug 14 2022, 3:31 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptAug 14 2022, 3:31 AM

Is there anything I can do as a random member of the public that wants this and knows C++?
My organization is trying to move to clangd for the default config, and the lack of docstring parsing is common feedback.

Is there anything I can do as a random member of the public that wants this and knows C++?

Maybe apply the patch locally, use it for a bit, and provide feedback?

I haven't forgotten about this review and I hope to get to it as time permits.

Is there anything I can do as a random member of the public that wants this and knows C++?

Maybe apply the patch locally, use it for a bit, and provide feedback?

I'm sure that would help!

I haven't forgotten about this review and I hope to get to it as time permits.

Would it maybe also help to split this patch into 2 or 3 smaller patches?