[PATCH] D64695: [clang-format] Added new style rule: SortNetBSDIncludes

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 15 01:28:36 PDT 2019


MyDeveloperDay added inline comments.


================
Comment at: unittests/Format/SortIncludesTest.cpp:709
+      "#include \"pathnames.h\"\n",
+      sort("#include <sys/param.h>\n"		
+           "#include <sys/types.h>\n"		
----------------
should you add a test which has groups defined already, I'm unclear as to how your algorithm works across groups?


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