[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