[PATCH] D57896: Variable names rule

Michael Platings via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 11 01:30:48 PST 2019


michaelplatings added a comment.

In D57896#1390517 <https://reviews.llvm.org/D57896#1390517>, @MyDeveloperDay wrote:

> Should we come up with a new style?  say `UpperOrLowerCamelCase`,   I don't mind going and doing that in the readability-identifier-naming check, given that I just wrote up all the Options for that check https://clang.llvm.org/extra/clang-tidy/checks/readability-identifier-naming.html in D56563: [clang-tidy]  add options documentation to readability-identifier-naming checker <https://reviews.llvm.org/D56563>


Sounds good to me. I see that you've made D57966 <https://reviews.llvm.org/D57966> a child of this issue, but we could swap the dependency around so that once your patch is applied I can update this patch to use `camelBackOrCase`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D57896





More information about the cfe-commits mailing list