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

Ronald Wampler via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 17 07:50:20 PDT 2019


rdwampler added a comment.

Thanks! Can you update the documentation too?



================
Comment at: lib/Tooling/Inclusions/HeaderIncludes.cpp:211
+      Ret = 0;
+    }
+  return Ret;
----------------
I think you can drop the else block and just return immediately from the for loop.


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