carlosgalvezp marked an inline comment as done. carlosgalvezp added inline comments.
================ Comment at: clang-tools-extra/docs/clang-tidy/checks/misc/use-anonymous-namespace.rst:7-9 +that could instead be moved into an anonymous namespace. It also detects +instances moved to an anonymous namespace that still keep the redundant +``static``. ---------------- bigdavedev wrote: > Would we want this to be a fix as well? Removing the `static` keyword > automatically would be nice. Would make sense to fix, but unfortunately with my limited AST knowledge I don't know how to get the location of the `static` token to apply a removal. So unless someone wants to help out with that it will have to stay as is for now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137340/new/ https://reviews.llvm.org/D137340 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
