[PATCH] D57896: Variable names rule
Roman Lebedev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 7 09:33:25 PST 2019
lebedev.ri added a comment.
In D57896#1389046 <https://reviews.llvm.org/D57896#1389046>, @michaelplatings wrote:
> In D57896#1389042 <https://reviews.llvm.org/D57896#1389042>, @lebedev.ri wrote:
>
> > Pretty sure this patch should have gone to llvm-commits, not cfe-commits.
>
>
> I just set the repository, Phabricator did the rest - apparently the magic isn't working so well.
1. Does clang-tidy warn on every single existing variable now?
2. It might be best to give this more visibility, by submitting a mail to llvm-dev, with a **noticeable** subject, like "RFC: changing variable naming rules in LLVM codebase"
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