lebedev.ri added a comment. In D64695#1585772 <https://reviews.llvm.org/D64695#1585772>, @Manikishan wrote:
> In D64695#1585754 <https://reviews.llvm.org/D64695#1585754>, @rdwampler wrote: > > > I am not quite sure why this change is required to sort the headers for > > NetBSD, you can set the priorities via `IncludeStyle.IncludeCategories`. Is > > that not sufficient? > > > It can be done by setting priorities in IncludeCategories, but here we have > nearly 40+ cases and categories to hardcode due to complex interdependencies > between their headers. So, I have added this style reducing the cases using > regex. And if this is fully parameterised any OS related project can add > their own header priorities. Note that `IncludeCategories` is already a regex - https://clang.llvm.org/docs/ClangFormatStyleOptions.html Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64695/new/ https://reviews.llvm.org/D64695 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
