[PATCH] D82604: No nested namespaces in Clang-Tidy checkers

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 25 18:35:22 PDT 2020


MaskRay requested changes to this revision.
MaskRay added a comment.
This revision now requires changes to proceed.

As the warnings suggest, nested name definitions can't be used because llvm-project is still using C++14.

`llvm.org` as a reviewer doesn't work. You can use `git log` and find active contributors for your patches.

Last time you closed a revision manually for a commit I pushed on your behalf - it is not necessary, Phabricator will close a revision automatically if a commit description contains `Differential Revision: `


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82604





More information about the cfe-commits mailing list