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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D53083: [clang-doc] ... Jake Ehrlich via Phabricator via cfe-commits

Reply via email to