MyDeveloperDay added inline comments.
================
Comment at: clang/unittests/Format/FormatTestCSharp.cpp:19
+
+class FormatTestCSharp : public ::testing::Test {
+protected:
----------------
klimek wrote:
> If everything's static, you don't need a test fixture at all, you can just
> make it plain functions and use TEST() instead of TEST_F().
I was actually just following the existing style of the other tests, but
looking into what you said it seems we could almost do this in the other tests
too.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58404/new/
https://reviews.llvm.org/D58404
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
