[PATCH] D57896: Variable names rule

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 21 14:01:45 PST 2019


probinson added a comment.

In D57896#1406353 <https://reviews.llvm.org/D57896#1406353>, @MyDeveloperDay wrote:

> I can't argue with anything you say.. but I guess to reinforce your point introducing what is effectively a 3rd style would likely cause even more jarring...


Zach isn't introducing a new style, the style already exists and is consistently used by what I think is our 3rd largest subproject.  It happens not to be used at all by the two largest subprojects, but those subprojects already aren't consistent with themselves.
I would not mind a more concerted effort to migrate to whatever style we pick, which was notably lacking last time around. Then the jarring inconsistencies would go away, and we could all get back to complaining about content and not style.


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