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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits