diff options
author | Angel Garcia Gomez <angelgarcia@google.com> | 2015-10-02 13:20:11 +0000 |
---|---|---|
committer | Angel Garcia Gomez <angelgarcia@google.com> | 2015-10-02 13:20:11 +0000 |
commit | 88d204442be24dd691ad95078e86565cb0321831 (patch) | |
tree | 586eee21a5af1960159faee568062b7a6ccbad56 /clang-tools-extra/test/clang-tidy/modernize-loop-convert-lowercase.cpp | |
parent | 911951f4f85487df14c9b8e87921e1fb96539b7c (diff) | |
download | bcm5719-llvm-88d204442be24dd691ad95078e86565cb0321831.tar.gz bcm5719-llvm-88d204442be24dd691ad95078e86565cb0321831.zip |
Handle trailing underscores on modernize-loop-convert variable namer.
Summary: https://llvm.org/bugs/show_bug.cgi?id=24961.
Reviewers: klimek
Subscribers: cfe-commits, alexfh
Differential Revision: http://reviews.llvm.org/D13381
llvm-svn: 249127
Diffstat (limited to 'clang-tools-extra/test/clang-tidy/modernize-loop-convert-lowercase.cpp')
-rw-r--r-- | clang-tools-extra/test/clang-tidy/modernize-loop-convert-lowercase.cpp | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/clang-tools-extra/test/clang-tidy/modernize-loop-convert-lowercase.cpp b/clang-tools-extra/test/clang-tidy/modernize-loop-convert-lowercase.cpp index 0b3d735e483..d1ca71aca58 100644 --- a/clang-tools-extra/test/clang-tidy/modernize-loop-convert-lowercase.cpp +++ b/clang-tools-extra/test/clang-tidy/modernize-loop-convert-lowercase.cpp @@ -7,6 +7,7 @@ const int n = 10; int arr[n]; int nums[n]; +int nums_[n]; void naming() { for (int i = 0; i < n; ++i) { @@ -23,6 +24,13 @@ void naming() { // CHECK-FIXES: for (auto & num : nums) // CHECK-FIXES-NEXT: printf("%d\n", num); + for (int i = 0; i < n; ++i) { + printf("%d\n", nums_[i]); + } + // CHECK-MESSAGES: :[[@LINE-3]]:3: warning: use range-based for loop instead + // CHECK-FIXES: for (auto & num : nums_) + // CHECK-FIXES-NEXT: printf("%d\n", num); + int num = 0; for (int i = 0; i < n; ++i) { printf("%d\n", nums[i] + num); |