[PATCH] D141787: [clang-tidy] fix a false positive of `modernize-concat-nested-namespaces`

Vincent Hong via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jan 15 05:16:40 PST 2023


v1nh1shungry added a comment.

In D141787#4054514 <https://reviews.llvm.org/D141787#4054514>, @v1nh1shungry wrote:

> This patch doesn't handle situations like:
>
>   namespace a {
>   namespace b {
>   #define FOO
>   namespace c {
>   }
>   }
>   }

It did. I must be mad :(


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141787



More information about the cfe-commits mailing list