[PATCH] D140532: .clang-tidy: disable misc-use-anonymous-namespace check.
Sam McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Dec 22 04:53:53 PST 2022
sammccall added a comment.
Yeah, good call. Do you mind if I first use this as a trigger to start a thread about this style rule on discourse, though? I've been meaning to, it came up in a review recently.
I think there's a fair chance of changing the policy from "require static" to "allow either" or "require anon namespace", and having the issue "live" in clang-tidy might make for a more engaged discussion.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D140532/new/
https://reviews.llvm.org/D140532
More information about the cfe-commits
mailing list