[cfe-dev] Identifier naming inconsistencies in clang
Richard via cfe-dev
cfe-dev at lists.llvm.org
Wed Jan 12 12:34:45 PST 2022
In article <SN4PR13MB5294C51D1BA806C35DFF725F92529 at SN4PR13MB5294.namprd13.prod.outlook.com>,
<paul.robinson at sony.com> writes:
> Some naming violations are deliberate; this is particularly true in
> llvm/include/ADT, where many classes define methods that intentionally
> follow the snake_case conventions used by STL. Those should not be
> changed.
Yeah, I noticed some of that too.
> You should feel empowered to update the conventions for code you are
> working on for other reasons (as separate NFC changes and not requiring a
> pre-commit review). But going through the entire project to update the
> conventions would probably be viewed as unnecessary churn, and potentially
> making life more difficult for people working on those areas.
I was definitely thinking of the former (stuff in clang-tidy and
related code like AST matchers) and not the latter.
--
"The Direct3D Graphics Pipeline" free book <http://tinyurl.com/d3d-pipeline>
The Terminals Wiki <http://terminals-wiki.org>
The Computer Graphics Museum <http://ComputerGraphicsMuseum.org>
Legalize Adulthood! (my blog) <http://LegalizeAdulthood.wordpress.com>
More information about the cfe-dev
mailing list