[PATCH] D137340: [clang-tidy] Add modernize-use-anonymous-namespace check

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 3 13:43:12 PDT 2022


njames93 added a comment.

I'm not entirely sure why this belongs in the modernize module given anonymous namespaces have been in c++ forever, maybe its more of a misc check? Also the modernize checks are meant to actually emit fixes(ignore the c arrays check :) ), Right now, this only warn, it doesn't appear to act


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137340/new/

https://reviews.llvm.org/D137340



More information about the cfe-commits mailing list