================ Comment at: unittests/cpp11-migrate/IncludeDirectivesTest.cpp:25 @@ +24,3 @@ + llvm::error_code EC = llvm::sys::fs::current_path(CurrentDir); + assert(!EC); + (void)EC; ---------------- Instead of using assertions, how about using `ASSERT_*` and then wrapping calls to `applyActionOnCode` with `ASSERT_NO_FATAL_ERROR()`. This is in keeping with the semantics of this function anyway: it should not fail.
http://llvm-reviews.chandlerc.com/D1438 BRANCH include-directives ARCANIST PROJECT clang-tools-extra _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
