[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