[libc-commits] [PATCH] D146048: [libc][NFC] Clean up clang-tidy warnings for `src/__support` and `src/math`.

[Siva Chandra via Phabricator via libc-commits]

sivachandra added inline comments.


================
Comment at: libc/.clang-tidy:12
   - key:             readability-identifier-naming.MemberIgnoredRegexp
-    value:           "^_[A-Za-z0-9_]+$"
+    value:           "_?(_[A-Za-z0-9]+)*"
   - key:             readability-identifier-naming.VariableCase
----------------
Why is this (and two more below) changed?


================
Comment at: libc/.clang-tidy:23
     value:           UPPER_CASE
+  - key:             readability-identifier-naming.LocalConstantCase
+    value:           UPPER_CASE
----------------
I suppose that if you don't add these, then the `VariableCase` complains about `UPPER_CASE` names. But, once you add this `LocalConstantCase`, it complains about `const` function args also and so you have to add `LocalConstantIgnoredRegexp`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146048