[PATCH] D57896: Variable names rule

Michael Platings via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 7 07:51:18 PST 2019


michaelplatings created this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

Following discussion <http://lists.llvm.org/pipermail/llvm-dev/2019-February/129854.html> and general agreement that the current naming rule for variables is not ideal, this patch switches the naming rule to make `lowerCamelCase` the standard, consistent with a prior RFC <http://lists.llvm.org/pipermail/llvm-dev/2014-October/077685.html>.

Given that over 450,000 variables are currently named in `UpperCamelCase`, the rule also permits using that form for consistency with existing code. I can't see a way to express that in .clang-tidy files.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D57896

Files:
  .clang-tidy
  clang/.clang-tidy
  llvm/.clang-tidy
  llvm/docs/CodingStandards.rst

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57896.185764.patch
Type: text/x-patch
Size: 4216 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190207/082cdc4d/attachment.bin>


More information about the cfe-commits mailing list