[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