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