We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
I built with a recent version of the clang-tidy
rnburn@localhost ~/bugs/ns-clang-tidy $ clang-tidy --version LLVM (http://llvm.org/): LLVM version 3.7.0svn Optimized build. Built Mar 30 2015 (04:29:52). Default target: x86_64-unknown-linux-gnu Host CPU: core-avx2
If I set up this file
//test.cpp namespace a { namespace b {
void f() { }
}}
Then run
clang-tidy -fix test.cpp -checks='*' -- -x c++ -std=c++1y -stdlib=libc++
It generates this file
//test.cpp -fixed namespace a { namespace b {
} // namespace b // namespace a
Note that it left out one of the closing braces for the namespace and breaks the source code.
The text was updated successfully, but these errors were encountered:
Thanks for the report and the proposed fix! I'll add a test and submit a fix shortly.
Sorry, something went wrong.
Fixed in r234403.
No branches or pull requests
Extended Description
I built with a recent version of the clang-tidy
rnburn@localhost ~/bugs/ns-clang-tidy $ clang-tidy --version
LLVM (http://llvm.org/):
LLVM version 3.7.0svn
Optimized build.
Built Mar 30 2015 (04:29:52).
Default target: x86_64-unknown-linux-gnu
Host CPU: core-avx2
If I set up this file
//test.cpp
namespace a { namespace b {
void f() {
}
}}
Then run
clang-tidy -fix test.cpp -checks='*' -- -x c++ -std=c++1y -stdlib=libc++
It generates this file
//test.cpp -fixed
namespace a { namespace b {
void f() {
}
} // namespace b
// namespace a
Note that it left out one of the closing braces for the namespace and breaks the source code.
The text was updated successfully, but these errors were encountered: