[PATCH] D75220: [clang-tidy] RenamerClangTidy now correctly renames `using namespace` decls

2020-03-18 Thread Nathan James via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG1365ab4b63b7: [clang-tidy] RenamerClangTidy now correctly renames `using namespace` decls (authored by njames93). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.or

[PATCH] D75220: [clang-tidy] RenamerClangTidy now correctly renames `using namespace` decls

2020-03-18 Thread Nathan James via Phabricator via cfe-commits
njames93 updated this revision to Diff 251190. njames93 added a comment. - Fixed comment nit Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75220/new/ https://reviews.llvm.org/D75220 Files: clang-tools-extra/clang-tidy/utils/RenamerClangTidyCheck

[PATCH] D75220: [clang-tidy] RenamerClangTidy now correctly renames `using namespace` decls

2020-03-18 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM aside from a commenting nit. Comment at: clang-tools-extra/clang-tidy/utils/RenamerClangTidyCheck.cpp:276 + +// fix using namespace decls +if (cons

[PATCH] D75220: [clang-tidy] RenamerClangTidy now correctly renames `using namespace` decls

2020-03-18 Thread Nathan James via Phabricator via cfe-commits
njames93 added a comment. Ping?? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75220/new/ https://reviews.llvm.org/D75220 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/