[PATCH] D142307: [clang-tidy][NFC] Use C++17 nested namespaces in clang-tidy headers

Carlos Galvez via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jan 22 11:01:10 PST 2023


carlosgalvezp added a comment.

In D142307#4071952 <https://reviews.llvm.org/D142307#4071952>, @Eugene.Zelenko wrote:

> Looks OK for me, but please fix small formatting issues. Will be good idea to await for other eyes.

Thanks for the quick review! I applied clang-format to the patch; any additional manual formatting wanted belongs in a separate patch IMHO.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142307



More information about the cfe-commits mailing list