[PATCH] D33932: [clang-format] Add support for case-insensitive header matching and use it to improve support for LLVM-style include sorting.

Eric Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 6 05:10:24 PDT 2017


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





More information about the cfe-commits mailing list