[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 Tue, 06 Jun 2017 05:10:57 -0700

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
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

Reply via email to