[PATCH] D44764: [clangd] Move GTest printers to separate file
Marc-Andre Laperle via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 22 12:26:54 PDT 2018
malaperle added a comment.
In https://reviews.llvm.org/D44764#1045682, @ilya-biryukov wrote:
> In https://reviews.llvm.org/D44764#1045648, @simark wrote:
>
> > We could create a file `ClangdTesting.h" which includes everything tested related (gtest, gmock, printers, syncapi, etc). The convention would be that test files would just include that.
>
>
> Yeah, sounds good. I would exclude syncapi from the list, though. Not all tests should necessarily depend on `ClangdServer`.
> @sammccall , @ioeric , @hokein, @bkramer are you on board with the idea to have an umbrella header for gtest+gmock+printers?
Sounds good to me too. I can update the patch once there is an agreement.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D44764
More information about the cfe-commits
mailing list