[PATCH] D117416: [clang-format] Handle C variables with name that matches c++ access specifier
MyDeveloperDay via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 17 23:57:30 PST 2022
MyDeveloperDay requested changes to this revision.
MyDeveloperDay added a comment.
This revision now requires changes to proceed.
I'm not a fan of this approach of adding a "C" language, mainly because of the `.h` problem so ultimately it doesn't solve your problem.
I think this is overkill for what is basically the subtle handling of "public/private/protected/try/delete/new" being used as variables (which frankly is at least in my view not a bright idea in the first place!)
I think ultimately we can try to catch those corner cases but IMHO it doesn't need the addition of a whole new language for that, (we are not at that much on an impasse)
This review contains .arclint that needs removing, this review contains no tests
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits