kadircet added inline comments.
================ Comment at: clang-tools-extra/clangd/test/semantic-tokens-refresh.test:13 +# CHECK: "method": "workspace/semanticTokens/refresh", +# CHECK-NEXT: "params": null +# CHECK-NEXT: } ---------------- lsp spec says `params: none` for this request. i am not sure if this is acceptable or if we should really change our transport layer to accommodate optional parameters. can't really test it on vscode atm unfortunately :/ Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97548/new/ https://reviews.llvm.org/D97548 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits