[PATCH] D64695: [clang-format] Added new style rule: SortNetBSDIncludes
Roman Lebedev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jul 17 10:33:37 PDT 2019
lebedev.ri added a comment.
In D64695#1589740 <https://reviews.llvm.org/D64695#1589740>, @Manikishan wrote:
> In D64695#1589508 <https://reviews.llvm.org/D64695#1589508>, @lebedev.ri wrote:
>
> > Is there sufficient test coverage as to what happens if `SortPriority` is not set?
>
>
> If SortPriority is not set, the Includes will be grouped without sorting,
Let me rephrase - for the exiting `.clang-format`s, that don't currently specify `SortPriority`,
this introduction of `SortPriority` should not change the header handling.
Is that the case, and if so is there sufficient test coverage for that?
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64695/new/
https://reviews.llvm.org/D64695
More information about the cfe-commits
mailing list