[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