[PATCH] D143691: Fix clang-formats IncludeCategory to match the documentation

Fabian Keßler via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 15 09:07:18 PST 2023


Febbe updated this revision to Diff 497701.
Febbe added a comment.

Added requested tests:

- since we now support priorities, equal to 0 (the main include priority) sorting should work now for those.
- multiple files can be main-includes now, the tests expects that now.
- negative priorities do not override the main include priority anymore, which is also tested


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D143691/new/

https://reviews.llvm.org/D143691

Files:
  clang/lib/Format/Format.cpp
  clang/lib/Tooling/Inclusions/HeaderIncludes.cpp
  clang/lib/Tooling/Inclusions/IncludeStyle.cpp
  clang/unittests/Format/SortIncludesTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D143691.497701.patch
Type: text/x-patch
Size: 11495 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230215/5fb2bf93/attachment-0001.bin>


More information about the cfe-commits mailing list