[PATCH] D53399: [clangd] Ensure that we reply to each call exactly once. NFC (I think!)
Sam McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 18 07:42:39 PDT 2018
sammccall created this revision.
sammccall added a reviewer: ioeric.
Herald added subscribers: cfe-commits, kadircet, jfb, arphaman, jkorous, MaskRay, ilya-biryukov.
In debug builds, getting this wrong will trigger asserts.
In production builds, it will send an error reply if none was sent,
and drop redundant replies. (And log).
No tests because this is always a programming error.
(We did have some cases of this, but I fixed them with the new dispatcher).
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D53399
Files:
clangd/ClangdLSPServer.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D53399.170084.patch
Type: text/x-patch
Size: 4869 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181018/b6d65675/attachment.bin>
More information about the cfe-commits
mailing list