[PATCH] D53083: [clang-doc] Add unit tests for merging
Jake Ehrlich via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 15 14:22:44 PDT 2018
jakehehrlich accepted this revision.
jakehehrlich added inline comments.
This revision is now accepted and ready to land.
================
Comment at: clang-tools-extra/unittests/clang-doc/MergeTest.cpp:222
+
+ Expected->DefLoc = Location(10, llvm::SmallString<16>("test.cpp"));
+ Expected->Loc.emplace_back(12, llvm::SmallString<16>("test.cpp"));
----------------
Use {} not () on both SmallString and Location (and any other constructors)
https://reviews.llvm.org/D53083
More information about the cfe-commits
mailing list