[PATCH] D40485: [clangd] Introduced a Context that stores implicit data
Ilya Biryukov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 11 04:53:26 PST 2017
ilya-biryukov added inline comments.
Comment at: clangd/Context.h:92
+ ContextBuilder derive() const &;
+ ContextBuilder derive() const &&;
> `&&`, not `const&&` :-)
> Maybe add a trailing `//takes ownership`
Right. Copy-paste is gonna kill me some day.
Added the comment, too. Although that should be obvious from the `&&` in the signature.
rCTE Clang Tools Extra
More information about the cfe-commits