Author: alexfh Date: Fri Oct 13 07:11:14 2017 New Revision: 315682 URL: http://llvm.org/viewvc/llvm-project?rev=315682&view=rev Log: [clang-tidy] Add a regression test for google-readability-namespace-comments
Add a regression test for the google-readability-namespace-comments bug introduced in r315057 (reverted in r315580). Modified: clang-tools-extra/trunk/test/clang-tidy/google-readability-namespace-comments.cpp Modified: clang-tools-extra/trunk/test/clang-tidy/google-readability-namespace-comments.cpp URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-tidy/google-readability-namespace-comments.cpp?rev=315682&r1=315681&r2=315682&view=diff ============================================================================== --- clang-tools-extra/trunk/test/clang-tidy/google-readability-namespace-comments.cpp (original) +++ clang-tools-extra/trunk/test/clang-tidy/google-readability-namespace-comments.cpp Fri Oct 13 07:11:14 2017 @@ -15,6 +15,20 @@ void f(); // So that the namespace isn't // CHECK-FIXES: } // namespace n2 // CHECK-FIXES: } // namespace n1 +#define MACRO macro_expansion +namespace MACRO { +void f(); // So that the namespace isn't empty. +// 1 +// 2 +// 3 +// 4 +// 5 +// 6 +// 7 +// CHECK-MESSAGES: :[[@LINE+2]]:2: warning: namespace 'macro_expansion' not terminated with +// CHECK-MESSAGES: :[[@LINE-10]]:11: note: namespace 'macro_expansion' starts here +} +// CHECK-FIXES: } // namespace macro_expansion namespace short1 { namespace short2 { _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits