[libcxx-commits] [PATCH] D114915: [libc++][NFC] Address clang-tidy warnings in include/

Arthur O'Dwyer via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Dec 2 08:42:17 PST 2021


Quuxplusone added inline comments.


================
Comment at: libcxx/.clang-tidy:2
 InheritParentConfig: true
-Checks: '-readability-identifier-naming'
+Checks: '-readability-identifier-naming,-llvm-header-guard,-llvm-include-order,-misc-unconventional-assign-operator'
----------------
I don't know if we're close to consensus one way or the other on the if-else/early-return refactorings, but IMHO this one-line diff in the `.clang-tidy` file is now uncontroversial and can be landed ASAP if you want. (Someone +1 that?)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114915



More information about the libcxx-commits mailing list