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

Chandler Carruth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 27 15:07:24 PDT 2017


chandlerc added inline comments.


================
Comment at: include/clang/Format/Format.h:993
+  /// inside ``IncludeCategories``.
+  bool IncludeRegexCaseInsensitive;
+
----------------
djasper wrote:
> Do we really need a flag here? Shouldn't we just always do this?
I have no opinion one way or the other. I opted to not change behavior for existing styles purely in the spirit of making as targeted of a change as I could..

Feel free to tell me how you want this to work / be tested.


https://reviews.llvm.org/D33932





More information about the cfe-commits mailing list