[PATCH] D18313: clang-format: Make include sorting's main include detection configurable.

Daniel Jasper via cfe-commits cfe-commits at lists.llvm.org
Sat Sep 10 01:26:31 PDT 2016


djasper closed this revision.
djasper marked 6 inline comments as done.
djasper added a comment.

Submitted as r263943.


================
Comment at: include/clang/Format/Format.h:415
@@ +414,3 @@
+  /// as the "main" include in both a.cc and a_test.cc.
+  std::string IncludeMainRegex;
+
----------------
klimek wrote:
> djasper wrote:
> > I chose this name for better alphabetical ordering. I don't strongly lean either way, WDYT?
> I'd still lean slightly towards MainIncludeRegex... but feel free to keep.
Renamed to IncludeIsMainRegex, which seems to be better than either of the other.


https://reviews.llvm.org/D18313





More information about the cfe-commits mailing list