ioeric added inline comments.
================ Comment at: unittests/Format/SortIncludesTest.cpp:269 + // Support case-sensitive and case insensitive matching. + Style.IncludeRegexCaseInsensitive = false; + EXPECT_EQ("#include \"GTest/GTest.h\"\n" ---------------- I think this TEST is only for main header sorting. Could you split tests for other categories into a separate TEST? https://reviews.llvm.org/D33932 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits