[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