[clang] [clang-tools-extra] [llvm] [clangd] introduce doxygen parser (PR #150790)

Maksim Ivanov via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 28 18:19:10 PDT 2025


================
@@ -1388,9 +1393,170 @@ static std::string formatOffset(uint64_t OffsetInBits) {
   return Offset;
 }
 
-markup::Document HoverInfo::present() const {
+markup::Document HoverInfo::presentDoxygen() const {
+  // NOTE: this function is currently almost identical to presentDefault().
----------------
emaxx-google wrote:

There's a lot of code that's duplicated here - this might eventually lead to subtle divergence between two branches and bugs. How big are the doxygen-specific bits going to be, and would it be possible to fit them into a single common implementation (maybe with a help of helper functions)?

https://github.com/llvm/llvm-project/pull/150790


More information about the llvm-commits mailing list