[PATCH] D57896: Variable names rule

Michael Platings via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 8 02:05:35 PST 2019


michaelplatings added a comment.

In D57896#1389067 <https://reviews.llvm.org/D57896#1389067>, @lebedev.ri wrote:

> 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"




1. Pretty much. Previously clang-tidy gave 56,787 unique warnings. After this patch it gives 361,382 unique warnings. I personally would be happy to change the settings from `camelBack` to `aNy_CasE`.

2. Done: http://lists.llvm.org/pipermail/llvm-dev/2019-February/130083.html


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D57896





More information about the llvm-commits mailing list