[PATCH] D140585: CodingStandards: restrict CamelCase variable names guideline to llvm/clang/clang-tools-extra/polly/bolt

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 5 13:31:09 PST 2023


probinson added a comment.

So, the real problem is, once somebody managed to herd the cats all into the same direction (or close enough), the consensus was "no mass conversion to the new style."  Which naturally left everything inconsistent.

I wasn't two years out of college before I was confronted by a page of code with something like 6 different styles on the same 24-line screen. Ever since (40 years!), I have understood that the critical thing is that _everybody uses the same style_, and it's much less important what that style actually is.  Trying to go in a direction of "there are different styles and we live with it" really doesn't help anyone.

Further discussion really belongs on Discourse, not in a code review.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140585



More information about the llvm-commits mailing list