alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.
LG. Thanks!
================
Comment at: unittests/Tooling/FixItTest.cpp:37
@@ +36,3 @@
+ EXPECT_EQ("foo(x, y)",
+ tooling::fixit::getText(CE->getSourceRange(), *Context));
+
----------------
Let's either put the tests in the `clang::tooling::<anonymous>` namespace (not
common for unit tests in this specific directory, but nevertheless reasonable,
imo) or add using declarations for `tooling::fixit::getText` and other tested
functions to reduce the amount of boilerplate.
http://reviews.llvm.org/D19941
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
