[PATCH] D70872: [clangd] Implement "textDocument/documentLink" protocol support

Michael Forster via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Nov 30 14:21:03 PST 2019


MForster marked an inline comment as done.
MForster added inline comments.


================
Comment at: clang-tools-extra/clangd/test/document-link.test:21
+# CHECK-NEXT:      },
+# CHECK-NEXT:      "target": "file://{{.*}}/iostream"
+# CHECK-NEXT:    },
----------------
Originally I tried to add a header file to the setup with a second didOpen request, but I didn't get this to work. Would I need to set this up like `background-index.test`, or is there a simpler way?

Anyway, I think the regular expression is probably good enough for the purpose of this test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70872





More information about the cfe-commits mailing list