diff options
author | Jonas Toth <jonas.toth@gmail.com> | 2018-09-25 18:12:28 +0000 |
---|---|---|
committer | Jonas Toth <jonas.toth@gmail.com> | 2018-09-25 18:12:28 +0000 |
commit | d1bd01c3183f537cbf63a71b8303a656b4c6cb65 (patch) | |
tree | 9816a0d6c6d61b66641093ea089ea619d8ad2dd3 /lldb/packages/Python/lldbsuite/test/expression_command/expr-in-syscall/TestExpressionInSyscall.py | |
parent | 02d600d267e5012eaabdf0e20afaf2dd7fb9b2e2 (diff) | |
download | bcm5719-llvm-d1bd01c3183f537cbf63a71b8303a656b4c6cb65.tar.gz bcm5719-llvm-d1bd01c3183f537cbf63a71b8303a656b4c6cb65.zip |
[clang-tidy] Add modernize-concat-nested-namespaces check
Summary:
Finds instances of namespaces concatenated using explicit syntax, such as `namespace a { namespace b { [...] }}` and offers fix to glue it to `namespace a::b { [...] }`.
Properly handles `inline` and unnamed namespaces. ~~Also, detects empty blocks in nested namespaces and offers to remove them.~~
Test with common use cases included.
I ran the check against entire llvm repository. Except for expected `nested namespace definitions only available with -std=c++17 or -std=gnu++17` warnings I noticed no issues when the check was performed.
Example:
```
namespace a { namespace b {
void test();
}}
```
can become
```
namespace a::b {
void test();
}
```
Patch by wgml!
Reviewers: alexfh, aaron.ballman, hokein
Reviewed By: aaron.ballman
Subscribers: JonasToth, Eugene.Zelenko, lebedev.ri, mgorny, xazax.hun, cfe-commits
Tags: #clang-tools-extra
Differential Revision: https://reviews.llvm.org/D52136
llvm-svn: 343000
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/expression_command/expr-in-syscall/TestExpressionInSyscall.py')
0 files changed, 0 insertions, 0 deletions